Public: anyone can read this; the crew can change it.
Asks come back when their cause changes even without a new check; every kind of ask has its own plural heading #770
Sign in to join this conversation.
Public: anyone can read this; the crew can change it.
Pillar: P1 Feed. Follow-up from the review of #692. One run, fresh from today's
main.Two small faults in how asks collapse and come back (
src/web/studio/asks.ts,src/web/studio/asks_page.ts):notDismissedtreats a dismissal of the row (ask:<kind>:<repoId>) as holding while the dismissal is newer than each item'ssince, which is its latest check. A pull that was stale (conflicting) when the row was dismissed, and merges again later without a new check, never comes back. Its cause did change (it can ship now), but the comparison misses it. The fix: give an ask'ssincethe moment it last became an ask. For a look ask that is the later of its latest green check and the moment the stale-ask sweep last saw it merge again. A dismissal holds only while it is newer than that.rowLine(asks.ts) androwHead(asks_page.ts) cover parked and stopped; any other kind gets "N green changes wait for your look". Declined and paused are one per repository today, so it does not show yet, but the fallback is wrong by construction. Give every kind its own plural, or refuse to collapse kinds that are one per repository.Acceptance.
DecisionKindhas a plural heading, checked by a test that walks the kinds./asksandGET /api/v1/asksshow the returned ask. The full check is green.Must not touch: the release workflow, the chart, other repositories.