-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[core] Update babel configs #9244
Conversation
// TODO fix useGridSelector side effect and enable back. | ||
// '@babel/plugin-transform-react-constant-elements', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@oliviertassinari I've uncommented this one to see if we can cleanup, not sure what's the side-effect though.
Linked: #666
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have updated the PR (#666) to better describe the issue.
Netlify deploy previewNetlify deploy preview: https://deploy-preview-9244--material-ui-x.netlify.app/ Updated pagesNo updates. These are the results for the performance tests:
|
// backport of https://github.com/zeit/next.js/pull/9511 | ||
['next/babel', { 'transform-runtime': { corejs: 2, version: transformRuntimeVersion } }], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Latest corejs version is 3
and this is 4 years old, I think we can use the default version?
Clean up the babel configs, and share as much as possible between the root & docs configs.
Linked: mui/material-ui#37461