From 945b58700f3b0afbd0362100ad648b262ae424fa Mon Sep 17 00:00:00 2001 From: Chris O'Neil Date: Mon, 28 Sep 2026 20:25:22 +0100 Subject: [PATCH] ci: run repo checks only on pull requests MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The push triggers on main and rc-* re-ran the exact checks that had just passed on the PR head. The organisation's runner pool is small enough that the duplicate work queues behind checks on open PRs, so the second run costs real review latency rather than just wasted minutes. - Drop the push trigger from ci.yml (MSRV, fmt, clippy, tests) - Drop the push trigger from adr-governance.yml pr-checks.yml, claude.yml and the tag-triggered release.yml are unchanged. release.yml carries no CI gate — its validate job only checks the tag against the Cargo.toml version. The default branch is main and nothing targets another branch, so no check loses coverage. Closes V2-1353 Co-Authored-By: Claude Opus 5 --- .github/workflows/adr-governance.yml | 9 --------- .github/workflows/ci.yml | 2 -- 2 files changed, 11 deletions(-) diff --git a/.github/workflows/adr-governance.yml b/.github/workflows/adr-governance.yml index e7911b4f..eff05ca8 100644 --- a/.github/workflows/adr-governance.yml +++ b/.github/workflows/adr-governance.yml @@ -7,15 +7,6 @@ on: - '.adr-kit.yaml' - 'scripts/adr-governance.py' - '.github/workflows/adr-governance.yml' - push: - branches: - - main - - master - paths: - - 'docs/adr/**' - - '.adr-kit.yaml' - - 'scripts/adr-governance.py' - - '.github/workflows/adr-governance.yml' jobs: adr-governance: diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 53890e90..3bf4bc96 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -1,8 +1,6 @@ name: CI on: - push: - branches: [main, "rc-*"] pull_request: branches: [main, "rc-*"]