feat(ci): add Slack failure notifications#151
Conversation
|
Stack: slack-notifications Part of a stacked PR chain. Do not merge manually. |
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughAdded a reusable GitHub Actions workflow Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
546e8d5 to
7d6c0cf
Compare
2503552 to
c6b93f8
Compare
|
f9e59a5 to
21ed234
Compare
Add scripts/slack.ts that posts block-kit formatted failure messages to a Slack incoming webhook. Extract a reusable notify-failure.yml workflow to keep the notification steps DRY across CI, stable release, and canary release pipelines. Requires a SLACK_WEBHOOK_URL repository secret.
Slack notifications should only fire for release failures, not on every pull request CI run.
Move the workflow-name input to a WORKFLOW_NAME env var so it is never interpolated into the shell. The --status flag remains as a hardcoded CLI argument since its value is always a literal.
21ed234 to
ba656ce
Compare
Railly
left a comment
There was a problem hiding this comment.
Looked through it, LGTM. Clean split between Slack for release pipeline failures and PR comments for snapshots. Nothing to flag, approve from me.
Summary
scripts/slack.tsthat posts block-kit formatted failure messages to a Slack incoming webhooknotify-failure.ymlworkflow to keep notification steps DRY across CI, stable release, and canary release pipelinesRequirements
Requires a
SLACK_WEBHOOK_URLrepository secret.Test plan
notify-failure.ymlis triggered when a CI job fails