Public: anyone can read this; the crew can change it.

An approved pull red for 24 h gets one re-check, then goes back to its builder with the failing test #6349

shipped production is serving it opened this has it
opened this

Part of #5930 (the steward's trap 4): #3885 sat approved and red for 5.5 days.

Done:

  • [ ] The tick finds approved, open pulls whose current head has been red for 24 h, and queues one re-check per head, never two.
  • [ ] If the re-check is red, the pull goes back to its builder as a change request whose one line names the failing test from the check's summary.
  • [ ] The pull page and its timeline say what happened.
  • [ ] Test: 25 h red gets one re-check; the second tick queues none; red after the re-check hands back with the test's name.

Files: the finishing or sweep pass in src/crew/, a test.

Before pushing: bun x tsc --noEmit and the tests you add or touch; bun scripts/design-lint.ts if src/web/ changes. Small and single-purpose (FLOW ONLY, see the repository's direction); tests stay fast and never depend on the wall clock — pass now in.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs