Skip to content

Prettier migration guide: mention ignore comments#1092

Merged
leaysgur merged 2 commits intooxc-project:mainfrom
slorber:patch-1
May 7, 2026
Merged

Prettier migration guide: mention ignore comments#1092
leaysgur merged 2 commits intooxc-project:mainfrom
slorber:patch-1

Conversation

@slorber
Copy link
Copy Markdown
Contributor

@slorber slorber commented May 7, 2026

Not sure how to include this exactly, but I though it's worth mentioning // prettier-ignore and // oxfmt-ignore comments here

As far as I understand oxfmt delegates to other tools like prettier as a fallback for certain kind of files, and // prettier-ignore comments remains relevant for them, for example .mdx files only support // prettier-ignore

Signed-off-by: Sébastien Lorber <slorber@users.noreply.github.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented May 7, 2026

Deploy Preview for oxc-project ready!

Name Link
🔨 Latest commit 72da5ea
🔍 Latest deploy log https://app.netlify.com/projects/oxc-project/deploys/69fd1b7f76378d000776f4d0
😎 Deploy Preview https://deploy-preview-1092--oxc-project.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread src/docs/guide/usage/formatter/migrate-from-prettier.md Outdated
Co-authored-by: Yuji Sugiura <6259812+leaysgur@users.noreply.github.com>
Signed-off-by: Yuji Sugiura <6259812+leaysgur@users.noreply.github.com>
Copy link
Copy Markdown
Member

@leaysgur leaysgur left a comment

Choose a reason for hiding this comment

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

(I've also included a link to the documentation, just in case, but) Thank you~!

@leaysgur leaysgur enabled auto-merge (squash) May 7, 2026 23:09
@leaysgur leaysgur merged commit c113ee5 into oxc-project:main May 7, 2026
7 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.

2 participants