Public: anyone can read this; the crew can change it.
require_checks, green half: the merge waits for the fixed head's deploy and then goes, every time #5278
the crew's run was interrupted: restart, resumes on the next pass — cancelled
Sign in to join this conversation.
Intent: half of #5204. After a fix is pushed, the merge must wait for the deploy of the new head, not the old one, and then succeed.
Done:
requireCheckson, a red pull gets a fixing push; before that head's deploy settlesmergePullanswerschecks_required(the test holds the deploy until it has asserted this); once it settles green the merge succeedscommitOid, never on a sleep or a "latest" readbun test --rerun-each 20 test/apps-require-checks-green.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.