Skip to content

Commit 847153e

Browse files
cpcloudclaude
andauthored
ci: use unique job names for result gates (#745)
## Summary - Rename gate jobs from the ambiguous `Result` to `CI Result`, `Lint Result`, and `Security Result` so each is individually selectable in branch protection rulesets - Suppress Codecov PR comments when coverage is unchanged (`require_changes: true`) 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 43656f1 commit 847153e

4 files changed

Lines changed: 5 additions & 5 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -265,12 +265,11 @@ jobs:
265265

266266
# ---------------------------------------------------------------------------
267267
# Gate: single required check that rolls up all jobs above.
268-
# After merging, update the repo ruleset to require only "CI / Result".
269-
# The job name below is just "Result"; GitHub prefixes the workflow name.
268+
# Ruleset requires "CI Result", "Lint Result", "Security Result".
270269
# ---------------------------------------------------------------------------
271270

272271
result:
273-
name: Result
272+
name: CI Result
274273
if: always()
275274
needs: [changes, test, benchmarks, nix-build, docs, semantic-release]
276275
runs-on: ubuntu-latest

.github/workflows/lint.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ jobs:
9191
run: nix run '.#pre-commit' -- --from-ref 'origin/${{ github.base_ref || 'main' }}' --to-ref HEAD
9292

9393
result:
94-
name: Result
94+
name: Lint Result
9595
if: always()
9696
needs: [changes, deadcode, golangci-lint, pre-commit]
9797
runs-on: ubuntu-latest

.github/workflows/security.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ jobs:
123123
uses: github/codeql-action/analyze@0d579ffd059c29b07949a3cce3983f0780820c98 # v4.32.6
124124

125125
result:
126-
name: Result
126+
name: Security Result
127127
if: always()
128128
needs: [changes, govulncheck, osv-scanner, secrets, codeql]
129129
runs-on: ubuntu-latest

codecov.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,3 +12,4 @@ coverage:
1212

1313
comment:
1414
layout: "files"
15+
require_changes: true

0 commit comments

Comments
 (0)