Skip to content

fix(ci): report skipped required checks safely#477

Merged
jeremi merged 3 commits into
mainfrom
agent/required-check-gates-20260725
Jul 25, 2026
Merged

fix(ci): report skipped required checks safely#477
jeremi merged 3 commits into
mainfrom
agent/required-check-gates-20260725

Conversation

@jeremi

@jeremi jeremi commented Jul 25, 2026

Copy link
Copy Markdown
Member

Summary

  • preserve the four existing branch-protection context names with lightweight always-running result jobs
  • run the expensive Rust, release-tooling, source-proof, and docs jobs only when their path classifiers require them
  • fail closed if path classification fails, a required job fails, or a job result is inconsistent with the classifier
  • add a regression test for the required context names and result contract

This removes the current failure mode where an intentionally skipped path-filtered job leaves an otherwise safe PR permanently blocked. It also avoids spending roughly an hour on unrelated build matrices.

Verification

  • python3 -m unittest release/scripts/test_registry_release.py (65 passed)
  • gate inventory plus release-tooling unit suites (110 passed)
  • actionlint .github/workflows/ci.yml
  • YAML parse
  • git diff --check
  • independent semantic review found no actionable issues

Signed-off-by: Jeremi Piotrowski [email protected]

@jeremi
jeremi enabled auto-merge (rebase) July 25, 2026 04:52
@jeremi
jeremi merged commit 8a1469c into main Jul 25, 2026
21 of 25 checks passed
@jeremi
jeremi deleted the agent/required-check-gates-20260725 branch July 25, 2026 05:18
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