Skip to content

ci: add Slack PR notification workflow#9

Open
stevefulme1 wants to merge 5 commits into
mainfrom
slack-pr-notifications
Open

ci: add Slack PR notification workflow#9
stevefulme1 wants to merge 5 commits into
mainfrom
slack-pr-notifications

Conversation

@stevefulme1

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

Sends alerts to Slack channel on PR open, close, merge, reopen, and review events.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Comment thread .github/workflows/slack-pr-notifications.yml Fixed
Comment thread .github/workflows/slack-pr-notifications.yml Fixed
Comment thread .github/workflows/slack-pr-notifications.yml Fixed
Comment thread .github/workflows/slack-pr-notifications.yml Fixed
Comment thread .github/workflows/slack-pr-notifications.yml Fixed
stevefulme1 and others added 4 commits April 15, 2026 15:12
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ntain permissions'

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
- Move all attacker-controlled values (PR title, branch name, user)
  into env vars instead of direct ${{ }} interpolation in JSON payload
- Use jq to build JSON payload safely, preventing injection via
  crafted PR titles or branch names containing quotes
- Pin slackapi/slack-github-action to commit SHA (v2.1.0)

@sabre1041 sabre1041 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@stevefulme1 Who is on point to set up the Webhook URL and add the Slack App? I can work with you to facilitate any actions from the CoP/GitHub side

@sabre1041

Copy link
Copy Markdown

@stevefulme1 Who is on point to set up the Webhook URL and add the Slack App? I can work with you to facilitate any actions from the CoP/GitHub side

Disregard the Webhook URL as it is already configured as a secret. What else is needed to be completed before integrating this enhancement?

@stevefulme1

Copy link
Copy Markdown
Contributor Author

Slack admin would need to approve the integration. After that its essentially capable. I tried to add it in slack and got shut down by the Admin automation overlords.

@sabre1041

Copy link
Copy Markdown

Slack admin would need to approve the integration. After that its essentially capable. I tried to add it in slack and got shut down by the Admin automation overlords.

Are we on hold for this PR until then?

@stevefulme1

Copy link
Copy Markdown
Contributor Author

i would pass it through so when/if they approve it then it will start workign right away

@sabre1041

Copy link
Copy Markdown

i would pass it through so when/if they approve it then it will start workign right away

Only concern is that it will cause failures until it is enabled

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