Skip to content

ci: accept headed changelog entries in the version-entry test - #121

Merged
snekxs merged 1 commit into
mainfrom
fix/changelog-entry-test
Nov 13, 2026
Merged

snekxs merged 1 commit into
mainfrom
fix/changelog-entry-test

Conversation

@snekxs

@snekxs snekxs commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

The changelog test in make-release-files.test.mjs required the version's entry to start with - . Since #115 the weekly entry starts with a ### New / ### Improvements / ### Fixes heading, so Bridge CI's "Helper and scripts" job failed on the 1.0.1 release commit.

The test now accepts an entry that starts with either a bullet or a ### heading. The release workflow itself is unaffected (its preflight passed).

Not run locally (no Node here); CI runs it.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Tests
    • Updated changelog validation to accept entries beginning with either a bullet or a ### heading.

The weekly changelog entry is now grouped under ### New, Improvements and
Fixes (#115), so it no longer starts with a bullet. The test that checks
the repo's own changelog entry still required "- " and failed on 1.0.1.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The changelog test now accepts entries that start with either - or ### .

Changes: Changelog Test

Layer / File(s) Summary
Accepted changelog prefixes
.github/scripts/make-release-files.test.mjs
The changelog-entry assertion accepts entries starting with - or ### .

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: Minimal · up to 40c19

This change updates a changelog test so headed weekly release entries pass. It has no production impact and is ready to merge.

Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title names the change: the version-entry test in the changelog tooling now accepts headed entries as well as bullet entries. It matches the raw summary, which says the test now accepts entries th…
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.
Full details: Title check

Explanation

The title names the change: the version-entry test in the changelog tooling now accepts headed entries as well as bullet entries. It matches the raw summary, which says the test now accepts entries that begin with either a bullet or a ### heading. The ci: prefix fits a change to a test script that runs in CI. The title does not cover the root cause, which is the weekly entry format, but it names the main change.

Finishing Touches

Generate unit tests (beta)

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

Recent review info

Run configuration

Commits

Reviewing files that changed from the base of the PR and between 1dd01b7 and 40c1902.

Files selected for processing: 1

  • .github/scripts/make-release-files.test.mjs

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

@github-actions github-actions Bot added type: ci PR title category: ci area: ci Automation and build configuration labels Nov 13, 2026
@snekxs
snekxs merged commit 248f47a into main Nov 13, 2026
18 checks passed
@snekxs
snekxs deleted the fix/changelog-entry-test branch November 13, 2026 20:18
snekxs added a commit that referenced this pull request Nov 13, 2026
The weekly changelog entry is now grouped under ### New, Improvements and
Fixes (#115), so it no longer starts with a bullet. The test that checks
the repo's own changelog entry still required "- " and failed on 1.0.1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci Automation and build configuration type: ci PR title category: ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants