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

Sentinel: shared:01m23vygyffx5hg2f5vm8r74sr:3ql98qate5 is past its budget: trevato/forge: "fork at the origin, and propose a change through it > proposing forks, branches on the fork, and opens … #4257

Merged opened this

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

Merged

scout/forge:scout/4232-sentinel-shared-01m23vygyffx5hg2f5vm8r74 → main 1 commit fork 1 thread · 1 unresolved

  • Merged. scout/4232-sentinel-shared-01m23vygyffx5hg2f5vm8r74 was merged into main as fea25a7.

Review

Sign in to review this change.
1 file changed +3 −1
test/web-workflows.test.ts +3 −1
@@ -404,7 +404,9 @@ describe("fork at the origin, and propose a change through it", () => {
404404 expect(sent.title).toBe("Fix the typo");
405405 const stored = fake.state.issues.get("octo/gears") ?? [];
406406 expect(stored[0]?.pull?.headRepo).toBe("me/gears");
407 });
407 // Fork, clone, commit and push over real git take ~2 s alone, which is
408 // over bun's 5 s default once the check machine is loaded (#4232).
409 }, 60_000);

Fine. A per-test timeout is the smallest fix. If a second real-git test needs it, move to a shared constant or the describe-level option.

408410
409411 test("a refused proposal re-renders the editor with every word still there", async () => {
410412 fake.state.refuseWrite = "no pulls today";

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs