Skip to content

ci: add Slack PR notification workflow#52

Merged
sabre1041 merged 2 commits into
mainfrom
test-workflow-branch
Jul 7, 2026
Merged

ci: add Slack PR notification workflow#52
sabre1041 merged 2 commits into
mainfrom
test-workflow-branch

Conversation

@burigolucas

@burigolucas burigolucas commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Adds a GitHub Actions workflow that sends Slack notifications on PR events (opened, closed/merged, reopened, and reviews)
  • Notifications go to the OVM Slack channel via incoming webhook
  • Requires SLACK_WEBHOOK_URL repo secret to be configured

Test plan

  • Add SLACK_WEBHOOK_URL secret to the repo
  • Open a test PR and verify Slack notification appears in channel
  • Verify merged PR notification works

🤖 Generated with Claude Code

Add GitHub Actions workflow that sends Slack notifications on PR events
(opened, closed/merged, reopened, and reviews). Uses the proven workflow
from openshift_virtualization_ops. Requires SLACK_WEBHOOK_URL repo secret.
@burigolucas burigolucas requested a review from sabre1041 as a code owner July 7, 2026 09:59
@burigolucas burigolucas closed this Jul 7, 2026
@burigolucas burigolucas changed the title Testing PR workflow ci: add Slack PR notification workflow Jul 7, 2026
@burigolucas burigolucas reopened this Jul 7, 2026
@burigolucas burigolucas closed this Jul 7, 2026
@burigolucas burigolucas reopened this Jul 7, 2026
@burigolucas burigolucas requested a review from stevefulme1 July 7, 2026 14:04

@sabre1041 sabre1041 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sabre1041 sabre1041 merged commit 9ccca89 into main Jul 7, 2026
64 of 65 checks passed
@sabre1041 sabre1041 deleted the test-workflow-branch branch July 7, 2026 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants