Skip to content

ci: make the governance pipeline PR-only - #870

Merged
robsongajunior merged 1 commit into
mainfrom
ci/governance-pr-only
Aug 7, 2026
Merged

ci: make the governance pipeline PR-only#870
robsongajunior merged 1 commit into
mainfrom
ci/governance-pr-only

Conversation

@robsongajunior

@robsongajunior robsongajunior commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove the `push: main` trigger from the Governance Pipeline: the Governance Gate is a required check with strict up-to-date enforcement, so the exact tree that lands on main already passed on the PR — the post-merge rerun was pure duplication.
  • Drop the release-please skip condition from ci: skip the governance rerun on release-please merges #869 (dead code without a push trigger) and simplify `concurrency` to the PR-only form.
  • Point the README badge at PR runs (`?event=pull_request`) — `?branch=main` would freeze on the last pre-change run forever.

How to test

  1. On this PR, open the Checks tab: the Governance Pipeline runs on the `pull_request` event and Governance Gate reports on the head SHA (the required check that gates the merge).
  2. After merging, watch the Actions tab filtered by `main`: no Governance Pipeline run appears for the merge push (before, every merge reran the full pipeline; ci: skip the governance rerun on release-please merges #869 only skipped release-please merges).
  3. Check the README badge: it links and renders from the latest `pull_request` run.

Notes

  • None

@robsongajunior
robsongajunior requested a review from a team August 7, 2026 20:00
@robsongajunior
robsongajunior requested a review from a team as a code owner August 7, 2026 20:00
@robsongajunior
robsongajunior force-pushed the ci/governance-pr-only branch from 3120c8c to e78fdeb Compare August 7, 2026 20:04
@robsongajunior
robsongajunior merged commit 653111f into main Aug 7, 2026
17 checks passed
@robsongajunior
robsongajunior deleted the ci/governance-pr-only branch August 7, 2026 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants