docs(772): the merge gate decides on the COMBINED state, and skipped counts as green
Build ErsatzTV Image / CI toolchain image resolves (pull_request) Successful in 8s
PR Gates / CI image pin matches docker/ci (pull_request) Successful in 15s
Build ErsatzTV Image / Delimiter ban (release path) (pull_request) Successful in 19s
PR Gates / Fix proofs (Proves trailers) (pull_request) Successful in 12s
PR Gates / Docs update reminder (pull_request) Successful in 34s
PR Gates / decisions lifecycle (pull_request) Successful in 23s
Review verdict / Set review-verdict status (pull_request_target) Successful in 11s
PR Gates / Script lint and tests (ruff + pytest) (pull_request) Successful in 5m0s
Build ErsatzTV Image / Build & test (.NET) (pull_request) Successful in 8m47s
Build ErsatzTV Image / EF migration integrity (SQLite + MySql) (pull_request) Successful in 6m21s
Build ErsatzTV Image / Build & push image (amd64) (pull_request) Skipped
Build ErsatzTV Image / Functional E2E (curl + UI contracts) (pull_request) Successful in 5m59s
Build ErsatzTV Image / API docs in sync (OpenAPI + endpoint index) (pull_request) Successful in 9s
Build ErsatzTV Image / Formatting (changed .cs conform to .editorconfig) (pull_request) Successful in 5s
review-verdict/h10 Review-verdict: MERGEABLE @ 78fc283 (base: main)

Graded a nit and not blocked on, but it is a claim about a neighbouring subsystem that is
one notch too strong: the hook decides on the combined `.state` and its filters exclude
`skipped` (#593). Left as-is it would have taught the next reader that any non-success
context denies, which is what #593 exists to correct.

refs #772
This commit is contained in:
2026-08-23 00:39:41 +02:00
parent 8c72a5de9e
commit 78fc283627
+3 -2
View File
@@ -1652,8 +1652,9 @@ registry blip does not redden a PR — that pause is what makes failing on unkno
shortening it silently trades this guard for flake.
**It is not a `needs:` of anything, but it is not consequence-free either.** The merge-consent hook
reads the PR's **combined** status and denies on any non-`success` context (ersatztv#598), so a red
preflight blocks the merge exactly like any other red job. "Advisory" would be the wrong word for
reads the PR's **combined** status and denies on a non-`success` combined state (a `skipped` context
counts as green, ersatztv#593; an advisory red does not, ersatztv#598), so a red preflight blocks the
merge exactly like any other red job. "Advisory" would be the wrong word for
it — what it does not do is *skip* the jobs it diagnoses.
**Recovery, without needing CI to be healthy.** The tag names a commit, and that commit still builds