-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
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
[utils] Allow nested imports in @mui/utils to speed up build #37586
[utils] Allow nested imports in @mui/utils to speed up build #37586
Conversation
Netlify deploy previewhttps://deploy-preview-37586--material-ui.netlify.app/ Bundle size report |
4b5a77c
to
879f54a
Compare
879f54a
to
a78c098
Compare
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.
LGTM
@flaviendelangle previously, a lot of the exports had the |
Do you have an import from the same endpoint that has changed? |
IMO we should discourage developers from importing from @mui/utils directly and treat this as an internal package. Individual functions that could be made public should be reexported from other packages. |
This :) what I mean is that in |
Part of #35840
See mui/mui-x#9172 for context
I also took the opportunity to migrate some files to TS
One step toward #35840