Conversation
✅ Deploy Preview for eslint-code-explorer ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe build configuration replaces Babel-based React compiler integration with ChangesReact compiler migration
Estimated code review effort: 2 (Simple) | ~10 minutes Suggested reviewers: Merge Risk: ⚪ Minimal · up to The build switches React compiler integration from Babel to the supported Oxc-based plugin path. Dependency and plugin resolution are consistent, so the change 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 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches🧪 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 |
|
The changes LGTM and I verified the compiled output matches what we get from Babel today. That said, I don't think we should do the migration yet. The |
Prerequisites checklist
AI acknowledgment
What is the purpose of this pull request?
Use the native React compiler pipeline instead of babel.
This should give a speedup and we can remove the babel dependency as we do not use it for other transformations.
What changes did you make? (Give an overview)
Applied the steps in the migration guide for vite:
oxc-transform-reactby passingcompiler: trueto the react vite pluginI could not use the latest version of
oxc-transform(0.149.0) as this gave peer dependency issues.Related Issues
None
Is there anything you'd like reviewers to focus on?
Summary by CodeRabbit