ci: dispatch PR validation manually; never queue on PR events - #1780
Merged
Merged
Conversation
GitHub queues a workflow run before evaluating job conditions, and the PR swarm applied every opt-in label: 80 runs again blocked beta publish after the label-only cut. Remove pull-request triggers entirely from CI, plugin/SDK validation, and hotpath workflows. Dispatch on the branch ref when wanted; CI runs Linux by default and explicit booleans add OS, hosts, or parity. Default-branch pushes run Linux only. Remove the now-dead PR dogfood job and run MCP conformance with the manually requested stock-host lane's shared CLI.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why the prior two cuts still failed
GitHub creates/queues a workflow run before evaluating job-level conditions. #1744 made drafts skip all jobs; the run objects still queued. #1773 made
ci-fullthe only triggering label; the automated PR swarm appliedci-fullto every PR. At 10:00: 173 open PRs, 80 queued CI runs, beta.45 publish blocked. I cleared 81 runs manually; publish immediately started and shipped all 9 assets.Only robust GitHub-native policy
No PR event trigger exists for expensive validation:
ci.yml: default-branch push +workflow_dispatch.plugin-validation.yml,sdk-conformance.yml: path-filtered default-branch push + dispatch.pr-run-cleanup(closed) and read-only release-PR integrity retain PR events.Run a PR head explicitly:
Opening, pushing, marking ready, and labels now create no run. A CI dispatch runs repository gates, harness self-tests, and Linux. Default-branch pushes run Linux only, so release matrix jobs retain runner slots. OS/host/perf are explicit.
More deletion
pr-dogfood: without PR context it could never run.debug-cliruns only withrun_hosts=true, its only remaining consumers.origin/masterand records branch metadata rather than empty PR fields.Net workflow diff: 65 additions, 272 deletions.
actionlinton all workflows, release safety, Rust cache-lineage guard pass.CONTRIBUTING.mdupdated.