Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Clarify Releases page is the canonical changelog source#49

Merged
wnstfy merged 1 commit into
mainfrom
docs/clarify-changelog-source
May 23, 2026
Merged

Clarify Releases page is the canonical changelog source#49
wnstfy merged 1 commit into
mainfrom
docs/clarify-changelog-source

Conversation

@wnstfy
Copy link
Copy Markdown
Contributor

@wnstfy wnstfy commented May 22, 2026

Summary

PR #44 made the weekly-release workflow auto-generate the CHANGELOG entry into each release tag's commit, not into main. So:

  • Releases page → current notes (auto-generated from merged PRs)
  • Release tarball's CHANGELOG.md → snapshot for that release
  • main's CHANGELOG.md → frozen at the last manually-touched state (by design — avoids needing a branch-protection bypass for the bot)

The README sentence at line 232 pointed at the in-tree CHANGELOG.md as the canonical source, which is misleading for anyone browsing the repo on GitHub. This updates the sentence to lead with the Releases page.

hermes integration test note

This PR is also a real-world test of wn-hermes' review path:

  • Touches README.md only → no CODEOWNERS hit
  • Branch protection now requires count: 1 approval
  • If hermes is reviewing, it'll approve and auto-merge will fire
  • If hermes isn't reviewing yet, the PR will sit at mergeStateStatus: BLOCKED and surface that gap

Test plan

  • Single-line README edit, no behaviour change
  • hermes posts an approving review
  • CI (scan, misconfig, lint) green
  • mergeStateStatus reaches CLEAN, auto-merge fires

PR #44 made the weekly-release workflow auto-generate the CHANGELOG.md
entry into each release tag's commit, not into main. The in-tree
CHANGELOG.md (what `git clone` shows) is therefore frozen at the last
manually-touched state, while the Releases page and each release
tarball's CHANGELOG.md carry the current state.

The README sentence at line 232 pointed at the in-tree CHANGELOG.md
as the canonical source, which is now misleading for anyone browsing
the repo on GitHub. Updated to lead with the Releases page link, with
the in-tarball CHANGELOG.md mentioned as the per-release snapshot.
@wnstfy wnstfy enabled auto-merge (squash) May 22, 2026 18:19
@github-actions
Copy link
Copy Markdown

Trivy misconfiguration findings (HIGH + CRITICAL)

✅ No HIGH or CRITICAL misconfigurations detected.

Copy link
Copy Markdown
Collaborator

@wn-hermes wn-hermes left a comment

Choose a reason for hiding this comment

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

Looks safe to merge.

Reviewed scope:

  • README.md only; no CODEOWNERS-sensitive paths touched.
  • Single documentation wording change: points users to the GitHub Releases page as the canonical current changelog source, while keeping CHANGELOG.md as the per-release tarball snapshot.
  • No code, workflow, permissions, secrets, dependency, or runtime behavior changes.
  • CI is green: scan, misconfig, CodeQL, and CodeQL Analyze all passed.

No security concerns found.

@wnstfy wnstfy merged commit 856432a into main May 23, 2026
4 checks passed
@wnstfy wnstfy deleted the docs/clarify-changelog-source branch May 23, 2026 04:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants