Keep the check step's no-second-review sentence whole
mesh/delivery merged: source.main moved to novox/hq@e65d0f8f2bbd
mesh/check being carried to cdfed7593e73 on e65d0f8f2bbd | from b229f5c4df1e
mesh/delivery merged: source.main moved to novox/hq@e65d0f8f2bbd
mesh/check being carried to cdfed7593e73 on e65d0f8f2bbd | from b229f5c4df1e
Claude-Session: https://claude.ai/code/session_01JG73KK8HD1gufnqyFxMFdW
This commit is contained in:
@@ -542,8 +542,8 @@ an answer no hq commit holds, is a finding of the review (step 2, item 2a).
|
||||
checker drops a check still running when its pull request merges (met twice on 2026-11-10): the lead reads the
|
||||
check's verdict (`mesh-delivery.show`) after every merge, and a merged change whose check has none is reported to
|
||||
the operator as merged unchecked, with what its local runs covered, never as passed. How it is checked: a report
|
||||
that says a merged change passed is read against its verdict in `mesh-delivery.show`. No
|
||||
second review: step 2 reviewed this tree.
|
||||
that says a merged change passed is read against its verdict in `mesh-delivery.show`. No second review: step 2
|
||||
reviewed this tree.
|
||||
3. **A check that fails** goes back to the same reviewer, continued by its id, with the failure, for **one more cycle** of step 2 with
|
||||
the same fixing agents; then push **once** again (the one more push of ADR 0548). The check is **read, never re-asked to see if it passes**: a
|
||||
check asked again without a new, reviewed head spends the build capacity for nothing. **The failure is read from the check's whole log**, never
|
||||
|
||||
Reference in New Issue
Block a user