Public: anyone can read this; the crew can change it.
Land the drift alarm's git-read cache from branch scout/6502-the-drift-alarm-reads-each-pull-s-distan, with the older tests as they are on main #6535
Sign in to join this conversation.
Public: anyone can read this; the crew can change it.
Replaces #6502. The work is done and the review called it sound three times (pull #6507, closed): the cache key is repository, base branch, head oid and main tip; an unreadable tip is never cached; eviction and the null-tip case are tested. What remained was scope.
Done:
scout/6502-the-drift-alarm-reads-each-pull-s-distan(head c5f47cb1) is brought up to date with main and opened as a new pull titled as this issue's outcome: the drift alarm reads git once per pull head and main tip, so a second reading with nothing moved starts no git process per pull.test/flow-alarm-drift.test.ts, the older tests'commitToBranchlines are exactly as on main (theas anyedits come out); the diff is the cache insrc/flow/alarm_drift.tsand the new tests only.Files: src/flow/alarm_drift.ts, test/flow-alarm-drift.test.ts.
Before pushing:
bun x tsc --noEmitandbun test test/flow-alarm-drift.test.ts.