Skip to content

chore(release): format the generated CHANGELOGs - #32

Merged
baseballyama merged 1 commit into
mainfrom
fix/release-format-changelog
Oct 9, 2026
Merged

baseballyama merged 1 commit into
mainfrom
fix/release-format-changelog

Conversation

@baseballyama

Copy link
Copy Markdown
Member

Summary

CI on main has failed its format check since the release PR (#18) was merged, because of the CHANGELOGs that changesets generated. This formats them and makes the release workflow format its own output from now on.

Motivation

changeset version puts a blank line between a bullet and its nested list. oxfmt removes that blank line. As a result:

  • the release PR's CI failed (run 37405698676);
  • after the merge, CI on main failed on CHANGELOG.md and packages/editor/CHANGELOG.md (run 37571744421).

Every future release PR would fail the same way. There is no linked issue; the maintainer reported the failure.

Changes

  • CHANGELOG.md and packages/editor/CHANGELOG.md: formatted with oxfmt. Only the blank lines before nested lists are removed; the text is unchanged.
  • New version-packages script (changeset version && oxfmt). The release workflow now uses it as changesets/action's version command, so the release PR is formatted when it is opened.

No changeset: nothing user-visible changes.

Testing

  • pnpm format:check reports no issues in tracked files. The only findings are local untracked .claude/worktrees files.
  • The workflow change takes effect on the next release PR. It runs the same oxfmt command that CI checks with.

Breaking changes

None

Checklist

  • I have read CLAUDE.md and followed the project's conventions.
  • I have added or updated tests for the change.
  • I have added or updated documentation where user-visible behavior changed.
  • If this is a breaking change, I have added a changeset / CHANGELOG entry and flagged it above.
  • I have re-read my own diff and removed dead code, debug prints, and stale comments.
  • If I used an LLM to draft this PR, I have verified each change myself, the PR represents real work that warrants a maintainer's review, and I am willing to defend each line in review.

🤖 Generated with Claude Code

changesets writes CHANGELOG.md in a shape oxfmt rewrites, so the release PR
and main after it failed CI's format check. Format the two CHANGELOGs on
main, and run oxfmt after `changeset version` in the release workflow so the
release PR is formatted from now on.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 9, 2026 04:26

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.

@baseballyama
baseballyama merged commit abd281d into main Oct 9, 2026
5 checks passed
@baseballyama
baseballyama deleted the fix/release-format-changelog branch October 9, 2026 04:28
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