Skip to content

docs(release-process): put the credit into a CHANGELOG-extracted body's source - #151

Merged
CybotTM merged 1 commit into
mainfrom
docs/changelog-credits
Sep 22, 2026
Merged

CybotTM merged 1 commit into
mainfrom
docs/changelog-credits

Conversation

@CybotTM

@CybotTM CybotTM commented Sep 22, 2026

Copy link
Copy Markdown
Member

Merging this adds one paragraph to the section on crediting contributors in references/release-process.md: where the release workflow builds the body from CHANGELOG.md, the credit goes into the CHANGELOG entry, because the extracted body carries only the @mentions the entries carry.

Why

netresearch/terraform-provider-ad extracts the new version's section with sed and publishes it as the body. Its entries credit nobody, so every release there fails release-notes-status.sh with MISSING CREDITS until someone edits the published body. v0.5.3 was never edited and still contains no @mention at all, checked on 2026-09-22. v0.5.4 needed the hand edit the same day.

The paragraph says to write the credit into the entry, so the extraction publishes it, and to add it with gh release edit --notes-file for a release already out. CHANGELOG.md records the change under Unreleased.

Assisted by claude-code:claude-opus-5-5 — Session

@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 9f9e8ff8-2b41-4547-be9d-1cd26dbf1eb8

📥 Commits

Reviewing files that changed from the base of the PR and between a926e63 and 3b375a9.

📒 Files selected for processing (2)
  • CHANGELOG.md
  • skills/github-release/references/release-process.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The change updates release documentation for workflows that build a release body from CHANGELOG.md. It states that contributor credit must be written in the CHANGELOG entry itself because the published release body only carries the @mentions present in that entry. It also documents the MISSING CREDITS failure in release-notes-status.sh, gives an inline credit example, and notes a manual fix for already published releases. A matching note was added to the unreleased changelog.

Priority: ➖ Normal

Change: Other · Severity of issue fixed: Medium

Merge Risk: ⚪ Minimal · up to 3b375

This documentation change clarifies contributor-credit guidance without changing release behavior in this repository. No actionable merge-blocking risk remains.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The pull request title directly and specifically describes the main change: documenting where to place contributor credits when a release workflow extracts the release body from CHANGELOG.md. The titl…
Description check ✅ Passed The pull request description is directly related to the changeset. It explains the motivation for the change, references specific examples (netresearch/terraform-provider-ad releases v0.5.3 and v0.5.4…
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
✨ Simplify code
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@CybotTM
CybotTM marked this pull request as ready for review September 22, 2026 20:07
Copilot AI lite review requested due to automatic review settings September 22, 2026 20:07
@github-actions github-actions Bot added documentation Improvements or additions to documentation skill labels Sep 22, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@CybotTM

CybotTM commented Sep 22, 2026

Copy link
Copy Markdown
Member Author

Self-review: 51070a9

Copilot could not review (its review is the quota error, on an earlier head) and CodeRabbit refused this head as rate-limited, so the author reviewed it.

  • The diff adds one paragraph to references/release-process.md and one Unreleased entry to CHANGELOG.md, nothing else.
  • Its two facts were checked on 2026-09-22: release-notes-status.sh reports MISSING CREDITS for netresearch/terraform-provider-ad v0.5.3 and its body contains no @ at all; v0.5.4 reported the same until its body was edited by hand that day.
  • The paragraph's advice matches the section it sits in: a bare @login, inline at the change.

Gate at this head: mergeState=CLEAN, 22 checks pass and 1 skipped of 23, 0 unresolved threads.

Assisted by claude-code:claude-opus-5-5 — Session

…'s source

Some release workflows extract the new version's CHANGELOG section with
sed and publish it as the release body; netresearch/terraform-provider-ad
does. That body carries exactly the @mentions the CHANGELOG entries carry,
and that repository's entries credit nobody, so every release there fails
release-notes-status.sh with MISSING CREDITS until the published body is
edited by hand. v0.5.3 was never edited and still contains no @mention at
all (checked 2026-09-22); v0.5.4 needed the hand edit the same day.

The section on crediting contributors now says to write the credit into
the CHANGELOG entry, so the extraction publishes it, and to add it with
gh release edit for a release already out.

Assisted-by: claude-code:claude-opus-5-5
Agent-Session: https://claude.ai/code/session_01GY5RqMQzgKbEvCALQ7r1ec
Agent-Host: 0493f0
Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>
@CybotTM
CybotTM force-pushed the docs/changelog-credits branch from 51070a9 to 3b375a9 Compare September 22, 2026 22:08
@sonarqubecloud

Copy link
Copy Markdown

@CybotTM

CybotTM commented Sep 22, 2026

Copy link
Copy Markdown
Member Author

Self-review: 3b375a9

Re-confirming the review above for the rebased head. git range-diff shows the commit unchanged (=) against the reviewed 51070a95; only the base moved, to the merge of #150. CodeRabbit reviewed this head and generated no actionable comments. Gate: mergeState=CLEAN, 22 checks pass and 1 skipped of 23, 0 unresolved threads.

@CybotTM
CybotTM merged commit 62065e4 into main Sep 22, 2026
24 checks passed
@CybotTM
CybotTM deleted the docs/changelog-credits branch September 22, 2026 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation skill

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants