Skip to content

ci: replace PR notify with main-only failure notification - #39

Merged
gmarzot merged 1 commit into
mainfrom
clean-notifications
Mar 8, 2026
Merged

ci: replace PR notify with main-only failure notification#39
gmarzot merged 1 commit into
mainfrom
clean-notifications

Conversation

@gmarzot

@gmarzot gmarzot commented Mar 8, 2026

Copy link
Copy Markdown
Contributor

Removes notify-ci-failure.yml which was firing on every build completion
(including PRs) and appearing as a skipped workflow run in the Actions tab.

Replaces it with notify-main-failure.yml that uses branches: [main]
on the workflow_run trigger — so it only ever fires when CI runs on the
main branch (push events). PR branch builds never match the filter, so
no noise in Actions tab or PR checks.

Notification coverage:

  • PR CI failure — no notification (align with moxygen approach)
  • Main CI failure — Slack + email (new notify-main-failure.yml)
  • Release failure/success — Slack + email (existing publish.yml, unchanged)

This change is Reviewable

Remove notify-ci-failure.yml (workflow_run on all builds, showing
skipped in Actions tab for every PR). Replace with notify-main-failure.yml
which uses branches: [main] filter so it only fires on push-to-main
CI failures — never triggered by PR branch builds at all.
@gmarzot
gmarzot merged commit 5faaa90 into main Mar 8, 2026
4 checks passed
@gmarzot
gmarzot deleted the clean-notifications branch April 12, 2026 02:23
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.

1 participant