Skip to content

fix(ci): skip platform matrix for docs-only PRs - #889

Merged
sanohiro merged 2 commits into
mainfrom
agent/docs-only-platform-ci-scope
Aug 25, 2026
Merged

fix(ci): skip platform matrix for docs-only PRs#889
sanohiro merged 2 commits into
mainfrom
agent/docs-only-platform-ci-scope

Conversation

@sanohiro

Copy link
Copy Markdown
Owner

Summary

  • classify true docs-only pull requests with the shared fail-closed pr-tier.sh predicate loaded from protected main
  • skip the Linux/macOS compiler matrix only for that exact tier while retaining full checks for compiled prose, deletions, unknown paths, stacked PRs, pushes, and manual runs
  • add Platform CI (required) as the stable aggregate for branch protection
  • make local docs-only preflight and CI consume the same predicate

Trust boundary

  • both database and platform scope classifiers fail closed for non-main PR bases
  • both load classifier machinery only from refs/remotes/origin/main
  • the first PR containing the new predicate runs the full matrix because the trusted base cannot yet prove the lighter tier

Verification

  • bash scripts/test-pr-workflow.sh
  • bash scripts/test-db-ci-scope.sh
  • ruby -e 'require "yaml"; YAML.load_file(".github/workflows/ci.yml")'
  • findings-fixed scripts/pre-pr.sh on e8e91c5f

After this PR merges, branch protection will replace the three platform-specific required contexts with Platform CI (required). The platform jobs still run on code PRs; true docs-only PRs receive the aggregate success without launching them.

@sanohiro
sanohiro marked this pull request as ready for review August 25, 2026 15:15
@sanohiro
sanohiro merged commit f57b986 into main Aug 25, 2026
12 checks passed
@sanohiro
sanohiro deleted the agent/docs-only-platform-ci-scope branch August 25, 2026 15:15
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