ci: credit the author on breaking-change changelog headings#62
Merged
Conversation
The changelog body template appends ` by @author` to every normal group entry but not to the "Breaking Changes" `####` headings — add the same guarded attribution there. Also sync CLAUDE.md's PR section with #56 (the PR-description changelog block is gone; `!` on the title is the whole per-PR mechanism). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
|
✅ PR title follows the Conventional Commits spec. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #62 +/- ##
=======================================
Coverage 94.92% 94.92%
=======================================
Files 27 27
Lines 5893 5893
Branches 5893 5893
=======================================
Hits 5594 5594
Misses 212 212
Partials 87 87 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
The `!` flag is already covered by the PR-title bullet above, and the rendering / hand-written-migration-notes facts live at their source in release-plz.toml's comments. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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
1 participant
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.
Summary
The changelog body template credits
by @authoron every normal group entrybut not on the "Breaking Changes"
####headings. This release will be thefirst to render that section (#55 / #59 are
feat!), so add the same guardedattribution there:
#### … {{ commit.message }}{% if commit.remote.username %} by @{{ commit.remote.username }}{% endif %}Also drops CLAUDE.md's stale breaking-changes bullet: it still described the
PR-description changelog block dropped in #56, the
!flag is alreadycovered by the PR-title bullet, and the rendering / hand-written-migration-
notes facts live at their source in release-plz.toml's comments.
Verification
commit.remote.usernameis the same variable the normal-entry line two linesbelow already uses, under the same guard. Rendering can't be verified locally
without GitHub-integrated release-plz — after this merges, release-plz will
regenerate the open release PR (#48), where the Breaking Changes headings can
be eyeballed before releasing.
🤖 Generated with Claude Code