Conversation
Four consecutive red pushes to main, every one of them Full-scope (windows/amd64). That is not four unlucky bugs -- it is the only outcome the trigger allowed. full-scope ran on push-to-main, manual dispatch, or a 'full-test' label that did not fire on the labeled event, so a platform bug could not be caught before merging even in principle. The one trigger that worked was the one that runs too late. Now it runs on every pull request. needs: validate still keeps the three runners idle while the fast gate is red, so the cost is about a minute on a PR that was going to pass anyway. A permanently red main costs more than that: it trains everyone to stop reading CI, and then it is not a signal at all.
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
1 participant
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.
mainhas been red on four consecutive pushes. Every one wasFull-scope (windows/amd64).That is not four unlucky bugs — it is the only outcome the trigger allowed.
full-scoperan on push-to-main, manual dispatch, or afull-testlabel that (until #2) did not fire on thelabeledevent. So a platform bug could not be caught before merging even in principle; the one trigger that worked was the one that runs too late.This makes it run on every pull request.
needs: validatestill keeps the three extra runners idle while the fast gate is red, so the cost is roughly one extra minute on a PR that was going to pass anyway. A permanently redmaincosts more than that — it trains everyone to stop reading CI, and then it stops being a signal at all.This PR is its own proof: the matrix runs on it.