Repository navigation
chore: replace prettier with oxfmt - #11939
Conversation
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. ⛔ Files ignored due to path filters (1)
⚙️ Run configuration
⛔ Files ignored due to path filters (1)
You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
⛔ Files ignored due to path filters (1)
📒 Files selected for processing (36)
💤 Files with no reviewable changes (2)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe repository replaces Prettier with Oxfmt for formatting commands and labeler YAML generation. It adds Oxfmt configuration and reformats existing source, test, and documentation code without changing the described types or runtime behavior. ChangesFormatting toolchain and migration
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Other Merge Risk: ⚪ Minimal · up to No actionable regression is established in this formatting-tool migration; it is mergeable after normal checks. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)✅ Passed checks (4 passed)Full details: Docstring CoverageExplanation Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 31 files. (3 skipped: 3 unsupported.) ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
|
|
View your CI Pipeline Execution ↗ for commit 385242d
☁️ Nx Cloud last updated this comment at |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
🚀 Changeset Version Preview2 package(s) bumped directly, 23 bumped as dependents. 🟩 Patch bumps
|
size-limit report 📦
|
ad252f3 to
32b77a3
Compare
|
@DogPawHat I verified the switch locally:
I don't think this needs to be stacked on #11938, though. Cherry-picking this commit onto |
Swap the root formatter from Prettier to oxfmt, keeping the existing style (no semicolons, single quotes, trailing commas, 80 columns). The config and ignore patterns were migrated with `oxfmt --migrate=prettier`. - Pin `oxfmt` and declare `svelte` at the root, since oxfmt needs `svelte/compiler` to format `.svelte` files and only lists it as an optional peer - Port `scripts/generate-labeler-config.ts` to oxfmt's `format()` API (output is identical to Prettier's) - Reformat 31 files where oxfmt still uses the pre-3.9 Prettier layout for union types that break onto their own line, plus one CSS-in-JS value Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
32b77a3 to
385242d
Compare
🎯 Changes
Stacked on #11938
Replace prettier with oxfmt
The options and ignores have been ported to the
.oxfmtrc.jsonfile (oxfmtprintWidthdefault is 100 characters, so preserving 80 for now). While the configuration with Prettier is the same, oxfmt seems to modify some things slightly differently, apparently due to the use of a pre-3.9 Prettier layout for union types that break onto their own line so we still end up with about 30 files changed.Files with actual changes are:
.prettierignore.prettier.config.js: Deleted. Replaced with.oxfmtrc.jsonpackage.json:formatscript changed tooxfmt. AddedoxfmtandsvelteasdevDependencies; removedprettierandprettier-plugin-svelte.pnpm-lock.yaml: Lockfile update for the dependency changes above.scripts/generate-labeler-config.ts: Formats with oxfmt'sformat()API instead of Prettier's. It reads.oxfmtrc.jsonitself, because the API doesn't find config files on its own, and throws if formatting reports errors. Its output is identical to what Prettier produced.✅ Checklist
pnpm run test:pr, or these tests do not apply to this pull request.🚀 Release Impact
Summary by CodeRabbit