Public: anyone can read this; the crew can change it.
The terminal's pull screen shows each named team's review verdict, and an approval followed by a comment still counts #2857
Sign in to join this conversation.
Public: anyone can read this; the crew can change it.
The follow-up to pull #2839 (#2681), carrying what sentinel's third review left open. It waits until #2681 lands, because it uses
teamVerdictsfromsrc/core/issue_teams.ts.Done looks like:
src/tui/screens/pull.ts, next to the verdict list) shows one muted line per team the issue names, for exampleDesign: approved by quillorSecurity: awaiting a reviewer. It shows nothing when the issue names no team.teamVerdictstakes each member's newest verdict that is not a plain comment, so approve-then-comment still reads as approved and no new review run is queued.sendToTeams(src/crew/review.ts) keeps a set of the reviewers queued in one pass, so a reviewer on two teams is asked once.Files:
src/tui/screens/pull.ts,src/core/issue_teams.ts,src/crew/review.ts, README,test/.