docs: transcribe the 13 outstanding CHANGELOG entries, and stop miscounting one - #806
Merged
Merged
Conversation
…unting one Release step 2 checks that `## Unreleased` is complete. It was not: `--verify-transcribed` found 13 merged PRs whose `## CHANGELOG entry` was written in the PR body and never copied into this file at merge, which is my omission as merger, not the authors'. #737, #743, #730, #738, #740, #741, #733, #754, #799, #797, #798, #804 and #800, transcribed here in merge order, newest first. Two needed a heading synthesised because their PR body gave the entry another shape: #730 wrote prose with no heading, and #754 wrapped a bullet in a ```markdown fence. `DECLARED_NONE` also gains a `#`-prefix allowance. #751 answered "no entry needed" as a HEADING (`### None, investigation only, no functional change`), and the anchored regex read that as a real entry naming a symbol called None. Nothing could ever transcribe it, so it would have reported MISSING forever. One new self-test row, proven load-bearing: reverting the allowance fails it and two neighbours. `--verify-transcribed` now reports 0 missing. The bare run still reports 37, which is correct and permanent: it asks what the merge commit did.
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.
Release step 2 is "check the
## Unreleasedsection is complete". It was not.Scripts/check-changelog-transcription.py --verify-transcribedreported 13 merged PRs whose## CHANGELOG entrywas written in the PR body and never copied intodocs/CHANGELOG.md. That ismy omission as merger, not the authors': changelog-on-merge
puts transcription at merge time, and I merged these without doing it.
Transcribed here in merge order, newest first: #800, #804, #798, #797, #799, #754, #733, #741,
#740, #738, #730, #743, #737. (#741 contributed three entries.)
Two needed a heading synthesised, because their PR body gave the entry another shape:
### \Curve3D.extrema(with:)` no longer crashes theprocess on parallel curves (Curve3D.extrema SIGSEGVs on parallel curves at every capacity, including its default #636)`.
```markdownfence. Given### Kernel pin moved to \v2.0.0-kernel.2`, carrying all fourteen patches (Rebuild OCCT.xcframework with patch 0017 (#484 null-context guard) and ship a patch release #512)`.One detector fix
DECLARED_NONEgains a#-prefix allowance. #751 answered "no entry needed" as a heading(
### None, investigation only, no functional change (#597)), and the anchored regex read that as areal entry naming a symbol called None. Nothing could ever transcribe it, so it would have reported
MISSING forever, in a report a human is meant to trust at release.
One new self-test row. Proven load-bearing: reverting the allowance fails it and two neighbours
(23/23 -> 20/23).
Result
The file's own TRANSCRIPTION NOTE is rewritten to match; it still described the earlier state (ten
transcribed, six declared-none, "expect it to keep reporting 31").
The bare run still reports 37 and always will. It asks whether the merge commit itself touched
the file, which is the right question at merge time and unanswerable afterwards without rewriting
history.
--verify-transcribedis the one to read for whether anything is genuinely missing.Why this is a PR and not a push
changelog-on-merge.mdhas the merger commit the transcription onto the base. The repositoryruleset makes
gate-scriptsa required check onrefactor/381-pass1b, which declines a directpush. Worth reconciling in the policy after the release; noted, not changed here.
CHANGELOG entry
None. This PR is the transcription; an entry describing it would be circular.
SemVer impact
NONE. Documentation and one gate-script detector fix. No public API is touched.