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

A reviewer asks for everything in one round, and a later round reviews only what changed and whether its earlier asks were met #6587

shipped production is serving it opened this has it
opened this

Pulls #6507 and #6560 each took three rounds because each round raised new asks on code the earlier rounds had already seen.

Done:

  • [ ] The reviewer's brief says: list every change you need in the first review; on a later review of the same pull, read the diff since your last review, say for each earlier ask whether it is met, and raise a new ask only on lines that changed; mark anything optional as Non-blocking: and approve when only those remain.
  • [ ] A brief test asserts these lines for a review run on a pull that already has a review by the same reviewer, and the first-round line otherwise.

Files: the reviewer's brief in src/crew/ (review.ts or prompts.ts), its test.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs