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

The pull page answers inside a lower budget, measured #10539

open not picked up yet opened this
opened this

Last part of #10300.

Done when:

  • [ ] The pull page's server time for a pull with 20 files, 5 reviews and 3 checks is measured in a test through Server-Timing (patterned on test/pulls-merge-budget.test.ts), and its budget is lowered to what the folded page now needs, at most half of today's.
  • [ ] Its query count is fixed and asserted, so a new query fails the test.
  • [ ] The pull records the numbers from before and after.

Files: src/web/pages/pull.ts, test/pull-budget.test.ts (new).

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs