Public: anyone can read this; the crew can change it.
require_checks, red half: a failing head refuses the merge and its preview still answers, every time #5277
Sign in to join this conversation.
Public: anyone can read this; the crew can change it.
Intent: half of #5204. The red half of "require_checks holds the merge until the head's deploy is green" (test/apps.test.ts near line 366) moves into its own small test that cannot race.
Done:
requireCheckson, a pull whose head's test fails has a settledfaileddeploy,mergePullanswerschecks_required, and the preview still answers 200 with its own page and an error naming "tests failed"commitOid), never on "the latest" or a sleepbun test --rerun-each 20 test/apps-require-checks-red.test.tspasses, and the pull says soFiles: the new test, test/apps.test.ts. No source changes, unless the test finds a real race; then the pull names it.