Skip to content

Commit a9f7465

Browse files
authored
ci: centralize Slack webhook onto org-level SLACK_WEBHOOK_OSS_ALERTS (#78)
## Summary Standardizes this repo onto the CopilotKit **org-level** secret `SLACK_WEBHOOK_OSS_ALERTS` (visibility `all`, posts to **#oss-alerts**), removing reliance on the repo-local bare `SLACK_WEBHOOK` secret. This is a **pure secret-NAME swap** — the repo-local `SLACK_WEBHOOK` value is the same #oss-alerts webhook, so there is **no behavior change** and **no channel change**. ## Change - `.github/workflows/notify-pr.yml`: `${{ secrets.SLACK_WEBHOOK }}` → `${{ secrets.SLACK_WEBHOOK_OSS_ALERTS }}` (1 line). The `SLACK_WEBHOOK` env-var alias name is retained, so the rest of the step's script is untouched. `publish.yml` already references `secrets.SLACK_WEBHOOK_OSS_ALERTS` and was intentionally left unchanged. ## Validation - `python3 -c "import yaml; yaml.safe_load(...)"` — valid - `actionlint` — pass - No bare `secrets.SLACK_WEBHOOK` references remain in `.github/workflows/` ## Follow-up The now-unused repo-local `SLACK_WEBHOOK` secret will be deleted after merge.
2 parents 4e2c4e2 + 14cf18a commit a9f7465

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/notify-pr.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ jobs:
1010
- name: Notify Slack
1111
if: github.actor != 'github-actions[bot]'
1212
env:
13-
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
13+
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK_OSS_ALERTS }}
1414
PR_TITLE: ${{ github.event.pull_request.title }}
1515
PR_URL: ${{ github.event.pull_request.html_url }}
1616
PR_AUTHOR: ${{ github.event.pull_request.user.login }}

0 commit comments

Comments
 (0)