Skip to content

docs(ci): the status-check rule was seen blocking a merge - #184

Merged
Nghaiz merged 1 commit into
developfrom
docs/branch-protection-observed
Aug 25, 2026
Merged

docs(ci): the status-check rule was seen blocking a merge#184
Nghaiz merged 1 commit into
developfrom
docs/branch-protection-observed

Conversation

@Nghaiz

@Nghaiz Nghaiz commented Aug 25, 2026

Copy link
Copy Markdown
Owner

PR #183 was the first thing the new required_status_checks rule applied to, and it behaved: while build-test (windows-latest) was still running, mergeStateStatus was BLOCKED against mergeable: MERGEABLE, flipping to CLEAN the moment the job finished.

That is worth recording, and so is what it is not. It shows a pending required check blocks. The case the rule exists for is a failing one, and those are not the same event — GitHub could treat a concluded failure differently from a not-yet-reported check. The page now says strongly indicated, not proven, and points at the cheap way to close it: notice the next genuine CI failure rather than manufacture one.

Same discipline the force-push row two sections down already uses.

…hat is not

The page recorded require-status-check as "configured, not proven". PR #183 is a
stronger datapoint than that and it should be written down: while
build-test (windows-latest) was still running, mergeStateStatus was BLOCKED against
mergeable: MERGEABLE, and it flipped to CLEAN the moment that job finished. The rule
is load-bearing, not a setting that saved and did nothing.

But it is a stronger datapoint, not the proof. What was observed is that a PENDING
required check blocks. The case this rule exists for is a FAILING one, and GitHub
could treat a concluded failure differently from a not-yet-reported check. Recording
the distinction rather than rounding it up, because rounding it up is how a gate
gets trusted for something it does not do -- the same failure mode as the
force-push row two sections down, which is honest about never having been tested.

Closed cheaply by noticing the next real CI failure; not worth manufacturing one.
@Nghaiz
Nghaiz force-pushed the docs/branch-protection-observed branch from f2607e1 to 532e717 Compare August 25, 2026 10:43
@Nghaiz
Nghaiz merged commit 9329392 into develop Aug 25, 2026
10 checks passed
@Nghaiz
Nghaiz deleted the docs/branch-protection-observed branch August 25, 2026 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant