Restore the two sections of the parity document that left the branch (#155) - #160
Merged
iderex merged 1 commit intoAug 21, 2026
Merged
Conversation
…155) This does not finish #155. One path group that commit removed is still missing. What was wrong. `d3edfc95b8526033c79cb26afe48282c2c090e32` took its tree from an older state of the default branch and landed on top of a newer one, under a message describing a change to how one workflow pin is commented. `docs/quality-parity.md` was among the seven paths it replaced, and the file went from 631 lines to 209. Two sections come back. The five declared check names the table gave no verdict to, which is the shape that costs #26 an answer rather than a line of prose, because a name the document leaves unjudged can be read as kept or as dropped with equal justice and the two readings build two different gates. And the whole of `## Which contexts arrive, and on which pull requests`, which is the walk of which contexts reach which commits. Three rows of the table above them come back with the same hunk, and one of the three is wrong on the default branch today rather than merely shorter. The `prettier` row there says the prose half is issue #50 and that nothing in this tree formats Markdown. Both halves have since stopped being true: gh issue view 50 --repo Flowfin/lab --json number,state --jq '"#\(.number) \(.state)"' #50 CLOSED git grep -n 'name: prose format' origin/main -- .github/workflows/ origin/main:.github/workflows/prose.yml:53: name: prose format This is an insertion rather than a blob restore, because #156 added a row to the parameter table well below both sections, so the hashes differ and the diff is what to read. What it says is that the file differs from the state before the removal by that row and the two paragraphs above it, and by nothing else: git diff --stat 90656ba HEAD -- docs/quality-parity.md docs/quality-parity.md | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) grep -c 'require_extra_approval_for_unattributed_changes' docs/quality-parity.md 1 What failure this prevents is #26 assembling a required set out of a document that declares five names and judges none of them, and a reader taking the table for the whole answer while the question of which contexts arrive on which pull requests is answered nowhere. Every command the restored text pastes was re-run against the current `origin/main` before this change, rather than trusted because it had once been true, and all of them still reproduce. One deserves naming: the paste reading `zizmor.yml` lines 76 to 84 reproduces because the comment removed from that file by the same commit is back, so it would have been stale had this landed first. Signed-off-by: Nils Lehnen <30603423+iderex@users.noreply.github.com>
iderex
deleted the
parity/the-two-sections-that-left-the-default-branch
branch
August 21, 2026 14:15
This was referenced Aug 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refs #155
What this changes
docs/quality-parity.md, and nothing else. Two sections come back, and thethree rows of
## The tablethat were replaced in the same hunk.The five declared check names the table gave no verdict to, under
## What this board adds that the target does not have. Three platform suiteentries,
vetandrequired contexts, each with kept or dropped and a reason,in the same terms the table uses.
The whole of
## Which contexts arrive, and on which pull requests, which isthe walk of which contexts reach which commits: that the command above it reads
a commit no pull request produced, that one name is reported twice on the same
commit, that two names come from a code-scanning upload rather than from a job,
that the upload is conditional while the step failing on findings is not, that
no workflow here narrows itself to some pull requests, and that one job can
start and still be unable to do its work.
How the absence arrived
d3edfc95b8526033c79cb26afe48282c2c090e32took its tree from an older state ofthe default branch and landed on top of a newer one, under a message describing
a change to how one workflow pin is commented. This file was among the seven
paths it replaced and went from 631 lines to 209.
This one is an insertion rather than a blob restore, because #156 added a row to
the parameter table well below both sections, so the two hashes differ on
purpose and the diff is what to read. What the diff says is that the file now
differs from the state before the removal by that row and the two paragraphs
above it, and by nothing else:
So nothing here was retyped and nothing #156 landed was dropped.
What failure it prevents
Issue #26 assembles the required set out of this document. A name this tree
declares and this document gives no verdict to can be read as kept or as dropped
with equal justice, so two readers build two different gates from the same page.
Five names were in that state. They are declared here and the restored section
judges each of them:
The second failure is a reader taking the table for the whole answer while the
question of which contexts arrive on which pull requests is answered nowhere.
That walk is #62's deliverable, and it had left the branch.
One of the three restored table rows is wrong on the default branch today rather
than merely shorter, which makes this more than a length repair. The
prettierrow there says the prose half is issue #50 and that nothing in this tree formats
Markdown. Both halves have stopped being true:
Every paste in the restored text was re-run
Restoring prose that was true when it was written is not the same as restoring
prose that is true now. So every command the restored text pastes was run
against the current
origin/mainand against the live platform before thischange, rather than trusted because it had once been true, and all of them
still reproduce. That includes the two run logs and every historical check-run
listing.
One is worth naming. The paste reading
zizmor.ymllines 76 to 84 reproducesonly because the comment the same removal took out of that file is back. Had
this landed first, that paste would have named the wrong lines.
The commands that read the tree at a moving reference, re-run in order:
and the one reading the target board:
On the size
This is 425 lines added, above the 400 the corpus carries as a cap, and it is
not a scope that was planned wrong. One property holds across every changed
byte: each is a byte the default branch held at
90656ba, put back by reversingone commit's hunk on this file, and the two commands under How the absence
arrived are how a reader checks that property instead of reading the diff.
Splitting it would produce two changes to one file where the second cannot land
until the first has, for a document the removal took out in one piece.
What was run
At
5078b9427b9d69ea281d7c1958d0c8fad1715a4a, on Windows, with no graphicalsession and as an ordinary user:
go build,go vetandgofmt -leach printed nothing, which is the passingresult for all three.
The one check in this tree that reads this file is the prose format, and it is
the reason the five declared names are pasted with their leading indentation
stripped, since
prose-carries-a-tabrefuses a tab in tracked Markdown:That check judges whitespace and characters. Whether the restored sentences are
true is not something it reads, and the re-runs above are what stands behind
them instead.
No test was skipped for needing elevation, and none was run with any.
What this does not do
It does not finish #155. One path group that commit removed is still missing,
LICENSEwith the## Licensesection ofREADME.md, and that one is #148.It does not touch the second question #155 raises, which is whether anything
here should refuse a merge that removes a tracked path without saying so in its
body. Nothing in this tree reads what a change removed, and restoring a file
does not change that.
It does not move #26 or #62. What both landed is this text; what was missing is
the text. The required set on the default branch is untouched and is still
empty, so nothing here is a report of a required context that failed to arrive.
No second person has read this change. The evidence above stands in place of
one, and that is a disclosure rather than an assurance.