Skip to content

fix: preserve release note formatting - #79

Merged
jxom merged 1 commit into
mainfrom
jxom/fix-release-formatting
Aug 4, 2026
Merged

fix: preserve release note formatting#79
jxom merged 1 commit into
mainfrom
jxom/fix-release-formatting

Conversation

@jxom

@jxom jxom commented Aug 4, 2026

Copy link
Copy Markdown
Member

Add a Prettier compatibility configuration and format the generated changelog so Changesets release PRs remain compatible with Vite+ formatting.

@vercel

vercel Bot commented Aug 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
frog Ready Ready Preview Aug 4, 2026 8:42am

Request Review

@pkg-pr-new

pkg-pr-new Bot commented Aug 4, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/frog@79

commit: 6777cca

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6777cca234

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .prettierrc.json
{
"printWidth": 100,
"semi": false,
"singleQuote": true

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Match Vite+'s trailing-comma default

When a changeset contains an embedded multiline function call or declaration, the locked Changesets formatter uses Prettier 2.8.8's trailingComma: "es5" default, while Vite+'s oxfmt uses "all". The version command can therefore still generate a CHANGELOG.md that pnpm exec vp check rejects, only for a different syntax than the example fixed here. Mirror trailingComma: "all" in this compatibility configuration, or derive both formatter configurations from one authoritative source.

AGENTS.md reference: AGENTS.md:L77-L77

Useful? React with 👍 / 👎.

@jxom
jxom merged commit 702c4ea into main Aug 4, 2026
9 checks passed
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