Conversation
|
There was a problem hiding this comment.
Pull request overview
Updates Slack notification behavior in deployment workflows so the Slack mention is only applied when the workflow fails, matching the stated intent for reduced noise during successful runs.
Changes:
- Add
mention_on: failureto thenotify-slackreusable workflow invocation for staging/release deployments - Add
mention_on: failureto thenotify-slackinvocation for master-staging deployments - Add
mention_on: failureto thenotify-slackinvocation for cloud deployments
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| .github/workflows/deploy-staging-and-prepare-release.yaml | Configure Slack notification mentions to trigger only on failure for staging + prepare-release flow. |
| .github/workflows/deploy-master-staging.yaml | Configure Slack notification mentions to trigger only on failure for master-staging deployments. |
| .github/workflows/deploy-cloud.yaml | Configure Slack notification mentions to trigger only on failure for cloud deployments. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6359 +/- ##
=======================================
Coverage 42.67% 42.67%
=======================================
Files 2517 2517
Lines 43696 43696
Branches 10341 10380 +39
=======================================
Hits 18647 18647
Misses 23719 23719
Partials 1330 1330 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
mention on slack only if workflow fails