Skip to content

fix(release): ignore generated changelog formatting - #24

Merged
urjitc merged 1 commit into
mainfrom
codex/ignore-generated-changelog
Jul 26, 2026
Merged

fix(release): ignore generated changelog formatting#24
urjitc merged 1 commit into
mainfrom
codex/ignore-generated-changelog

Conversation

@urjitc

@urjitc urjitc commented Jul 26, 2026

Copy link
Copy Markdown
Member

Summary

  • exclude the Release Please generated changelog from repository formatting

Why

Release Please emits valid changelog Markdown whose spacing differs from the repository formatter. The generated 0.4.1 PR therefore fails pnpm check even though its package versions and release content are correct.

Testing

  • pnpm check
  • formatter check against the generated 0.4.1 release branch

Review Notes

.prettierignore is the formatter standard ignore file and contains only CHANGELOG.md.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.


Summary by cubic

Add .prettierignore to exclude CHANGELOG.md from Prettier. This avoids formatting conflicts with release-please-generated changelogs and unblocks pnpm check on release PRs.

Written for commit e3cffb6. Summary will update on new commits.

Review in cubic

Summary by CodeRabbit

  • Chores
    • Excluded the changelog from automatic formatting checks.

@cursor

cursor Bot commented Jul 26, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@coderabbitai

coderabbitai Bot commented Jul 26, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: ffaa9f07-7279-45ed-ba51-449f2c71f167

📥 Commits

Reviewing files that changed from the base of the PR and between 0ed5f3a and e3cffb6.

📒 Files selected for processing (1)
  • .prettierignore

📝 Walkthrough

Walkthrough

The Prettier ignore configuration was updated to exclude CHANGELOG.md from formatting.

Changes

Formatting configuration

Layer / File(s) Summary
Update Prettier ignore rules
.prettierignore
Adds CHANGELOG.md to the files skipped by Prettier.

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

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the change to ignore the generated changelog during formatting checks.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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)
  • Create PR with unit tests
  • Commit unit tests in branch codex/ignore-generated-changelog

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.

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 1 file

Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Jul 26, 2026

Copy link
Copy Markdown

Greptile Summary

Excludes generated changelog Markdown from formatting checks.

  • Adds CHANGELOG.md to the repository’s Prettier-compatible ignore file.
  • Prevents Vite+/Oxfmt formatting checks from rejecting Release Please changelog spacing.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The formatter supports .prettierignore, and the added pattern directly excludes generated changelog Markdown as intended without affecting application behavior.

T-Rex T-Rex Logs

What T-Rex did

  • T-Rex ran the pre-change formatter check and reported an exit code of 1 due to incorrect formatting in CHANGELOG.md.
  • T-Rex ran the formatter after applying changes, which exited with code 0 and showed the changelog was excluded, while the SHA-256 checksum remained unchanged.
  • T-Rex ran the pre-change complete check and reported an exit code of 1 due to the changelog formatting issue.
  • T-Rex ran the post-change complete check and reported an exit code of 0 with 251 files correctly formatted and 200 files having no lint warnings, lint errors, or type errors.

View all artifacts

T-Rex Ran code and verified through T-Rex

Important Files Changed

Filename Overview
.prettierignore Adds the generated changelog to the formatter ignore list; no actionable issue identified.

Reviews (1): Last reviewed commit: "fix(release): ignore generated changelog..." | Re-trigger Greptile

@urjitc
urjitc merged commit 8f59a7b into main Jul 26, 2026
10 checks passed
@urjitc
urjitc deleted the codex/ignore-generated-changelog branch July 26, 2026 18:55
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.

1 participant