Repository navigation
Put every CHANGELOG.md release header in the ## KiteViewers vX.Y.Z YYYY-MM-DD form and fold the stale 2024 ## Unreleased into v0.4.20 - #59
Conversation
Release headers become '## KiteViewers vX.Y.Z YYYY-MM-DD', '### Changes' becomes '### Changed', v0.4.24 carries its tag date, and the stale 2024 '## Unreleased' is folded into v0.4.20, the first release that shipped it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1-Bort-1
left a comment
There was a problem hiding this comment.
Independent review (advisory)
Verdict: APPROVE WITH COMMENTS · 0 inline, 0 off the diff
Good
- All 22
##lines now match## KiteViewers vX.Y.Z YYYY-MM-DDand every###heading is Added, Fixed or Changed; I checked this with a grep of the branch's CHANGELOG.md. - The v0.4.24 date fix (2025-01-06 → 2025-06-01) is correct:
git taggives the v0.4.24 creator date as 2025-06-01, and the v0.4.20–v0.4.23 dates also match their tags. - Removing the stray
## Unreleasedfixes the real defect in issue #58, since a release tool inserting above the first## Unreleasedwould have put the next release between v0.4.14 and v0.4.13. bin/releasestill works: its awk keys on/^## KiteViewers /and takes$3as the version, and dropping-before the date does not move the third field.- Dropping the duplicate 'add example
reelout_1p.jl' line is right, because v0.4.20### Addedalready has the same line. - Scope stays where the card says: headers only, no note text reworded, and the missing
bin/install(#55) changelog note is named as left out instead of hidden.
Not good
- CLAUDE.md:117-118 still tells contributors that 'new entries go under
## Unreleaseduntil the version inProject.tomlis bumped'. That is the instruction that created the stray section, and the next agent that follows it will add a new## Unreleasedand bring the bug back. It is close to this change and should be updated to thechangelog.d/convention in this PR or a linked follow-up. - PlanV3.md:150 likewise tells the reader to file entries under
## Unreleased. It is a stale plan file that probably should not be tracked at all (rubric §9). - Moving the three orphaned lines under v0.4.20 relies on commit dates and tag contents, not on the author. The evidence is sound, but the release notes now credit v0.4.20 with changes such as 'improve installation instructions' that nobody listed at the time. That is acceptable, but Uwe should confirm it as well as the v0.4.24 date.
- v0.4.13 and v0.5.2 keep fixes ('fix tests', 'fix examples', 'fix pre-compilation') under
### Changed. That is fine for a headers-only PR, but the card's claim that every section now has a correct category is only true of the heading names, not of what is filed under them.
claude, rubric CLEAN_CODE.md. A different lab from the implementer
on purpose: a reviewer sharing its blind spots would not flag its mistakes.
…er form Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Local full suite: PASS (3 min, Julia 1.13.0, one cell of the matrix) |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
CLAUDE.md: fixed in 14e0cca, its changelog rule now points at changelog.d/ and the release header form. PlanV3.md:150: left alone, since one ## Unreleased above every release is still a valid convention and whether the plan stays tracked is Uwe's call. v0.4.20 fold and v0.4.24 date: agreed, and the card's Risk line now asks Uwe to confirm both. Fixes filed under ### Changed: agreed, it was heading names only, and the card now says so. |
|
From Per 1-Bart-1/Agents#635: since 1-Bart-1/Agents#634 the release bump writes headers as |
Per 1-Bart-1/Agents#634 the release bump writes ## <Package> vX.Y.Z DD-MM-YYYY. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
## KiteViewers vX.Y.Z YYYY-MM-DD form and fold the stale 2024 ## Unreleased into v0.4.20## KiteViewers vX.Y.Z DD-MM-YYYY form and fold the stale 2024 ## Unreleased into v0.4.20
|
From Per 1-Bart-1/Agents#635 (comment 6099024080) ISO is the convention after all: the bump goes back to writing |
Undoes 16f4573: per 1-Bart-1/Agents#635 the release bump writes ## <Package> vX.Y.Z YYYY-MM-DD again. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
## KiteViewers vX.Y.Z DD-MM-YYYY form and fold the stale 2024 ## Unreleased into v0.4.20## KiteViewers vX.Y.Z YYYY-MM-DD form and fold the stale 2024 ## Unreleased into v0.4.20
|
Reverted in 7b06463: every CHANGELOG.md header and CLAUDE.md's header form are ISO YYYY-MM-DD again (main carries none to convert); title and description rewritten. No CITATION.cff in this repo. |
TL;DR
Every release header now reads
## KiteViewers vX.Y.Z YYYY-MM-DD, the form the release bump writes (1-Bart-1/Agents#635), and the file has no## Unreleasedleft. The old one sat between v0.4.14 and v0.4.13, so the nextagent releasewould have written the new release there instead of above v0.6.0.0.6.1 was never released
Project.tomlonmainsays0.6.0, not 0.6.1. That matches the latest GitHub release and tag (v0.6.0, 2026-08-13) and the top changelog header. Nov0.6.1tag exists, and no commit has ever setversion = "0.6.1"inProject.toml. The only change since v0.6.0 that a user could notice is thebin/installrewrite in #55, which has no changelog note yet. It needs achangelog.d/file before the next release has anything to say. I left that out of this PR, which only touches headers.Where the stale
## UnreleasedbelongedUwe wrote it on 2024-11-02 (8af477f), and he added
examples/reelout_1p.jlthe same day (c0abac8). v0.4.20 (2024-11-09) is the first tag that containsreelout_1p.jlandreelout_4p.jl, while v0.4.19 still hasreelout.jl. So those notes shipped in v0.4.20 but were filed under the wrong heading. Its three other lines move unchanged under v0.4.20's### Changed. I dropped its "add examplereelout_1p.jl" line because v0.4.20's### Addedalready has it, word for word.Other header fixes
v 0.5.2→v0.5.2, and the-before each date is removed.2025-01-06. Its tag and GitHub release are both 2025-06-01, so the day and month were swapped, and the header now says 2025-06-01.### Changes(v0.5.0–v0.5.2) →### Changed. v0.4.16 had no heading at all and now has### Fixedabove its one bugfix line.### Changed.CLAUDE.md
Its coding-style section told contributors to add new entries under
## Unreleaseduntil the version is bumped, which is how the stale section came about. It now points atchangelog.d/<issue>-<slug>.mdand the release header form.PlanV3.md:150still says## Unreleased; I left it, since one## Unreleasedabove every release is still a valid convention and whether that plan stays tracked is Uwe's call.Verification
withRelease(src/bump.ts) onmain's CHANGELOG with a probe fragment for v0.6.1. It put## KiteViewers v0.6.1at line 135, between v0.4.14 and v0.4.13. On this branch it lands at line 3, above v0.6.0.rules/release --dry-run --sha 7b06463(the box copy, run from the worktree) on the pushed head readsVersion: v0.6.0and prints the v0.6.0 notes.##line in CHANGELOG.md matches^## KiteViewers v[0-9.]+ [0-9]{4}-[0-9]{2}-[0-9]{2}$, and every###heading is Added, Fixed or Changed. I checked both with grep.bin/releasestill reads the version as its third field (v0.6.0).main. CI on 7b06463 not yet read.agent ci-local. This PR only changes markdown, and the box can't install this repo at the moment:./bin/install -yrefuses the box's default Julia 1.13. I filed that as 1-Bart-1/Agents#629, since the fix is the box's choice of Julia channel, not this repo.Scope
+29 / −31 across
CHANGELOG.mdandCLAUDE.md. InCHANGELOG.md(+26 / −29) most of it is the rewritten headers. The net −3 is the 7-line## Unreleasedblock removed, minus its 3 lines re-added under v0.4.20 and the one new### Fixed. No note text was reworded.CLAUDE.md(+3 / −2) is the changelog rule above.Opened by
1-Bort-1, an AI agent working for @1-Bart-1.Closes #58 · task
KiteViewers.jl-58