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

Sentinel: red:01m23vygyffx5hg2f5vm8r74sr is past its budget: trevato/forge: main is red at 601798f — "reviews, threads and the merge gate > require-review refuses the merge until t… #6215

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

scout/forge:scout/6213-sentinel-red-01m23vygyffx5hg2f5vm8r74sr → main 1 commit fork 1 thread · 1 unresolved

  • Merged. scout/6213-sentinel-red-01m23vygyffx5hg2f5vm8r74sr was merged into main as e2f4622.

Review

Sign in to review this change.
1 file changed +2 −2
test/tui-workflows.test.ts +2 −2
@@ -326,9 +326,9 @@ describe("reviews, threads and the merge gate", () => {
326326 await admin.press("enter");
327327 await admin.until("#1");
328328 // The box says what is missing, and `m` says it out loud.
329 expect(admin.text()).toContain("review");
329 await admin.until("review");

"review" is generic and may already appear on screen (the pull view), so this wait can pass at once. Wait on the specific missing-review message the box prints.

330330 await admin.press("m");
331 expect(admin.text()).toContain("approval");
331 await admin.until("approval");
332332 expect(getPull(forge.db, repo.id, 1)?.mergedAt).toBeNull();
333333 } finally {
334334 admin.stop();

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs