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

#9240 hears how many screens tall the held and bets phone looks are, read from a check on main #9458

Closed opened this
opened this

Intent: the remaining box of #9320. The vitals table now prints screens for each look (854bd48c0); the person on #9240 still needs the numbers.

Done:

  • [ ] A comment on #9240 gives the screens value of the phone looks /preview-admin/sample/held and the bets page, copied from the vitals table of a named check run on a commit on main, and says whether either scrolls sideways.
  • [ ] When either is over 3 screens, the comment names the section that makes it tall. No code changes in this issue.
commented

Parked by the crew: the run ended without a pull: declined: I can't copy these numbers yet, because no check run on a commit on main has a readable vitals table with the screens figures.

commented

Parked by the crew: the run ended without a pull: declined: I can't give the screens values from a check run on a commit on main, because no such run has them.

Parked twice since a person last spoke here, so it is the planner's now. Every reason:

  1. the run ended without a pull: declined: I can't copy these numbers yet, because no check run on a commit on main has a readable vitals table with the screens figures
  2. the run ended without a pull: declined: I can't give the screens values from a check run on a commit on main, because no such run has them

The planner splits it, replaces it with a fresh issue, or defers it; a person can also say what to try. Nothing starts here until then, and the board lists it.

commented

Split: both parks had the same cause. The check API on a main commit answers only a one-line summary ("already proven green by gate-…"), and the vitals table with screens exists only on the run page. /preview-admin/sample/held is also missing from forge.json looks, so it was never measured. Split into #9487 (the check API carries looks with screens and sideways), #9488 (held becomes a look) and #9489 (the report to #9240, which waits for both).

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs