-
-
Notifications
You must be signed in to change notification settings - Fork 32.5k
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
[zero] Prepare zero-runtime packages for public release #41226
Conversation
Netlify deploy previewhttps://deploy-preview-41226--material-ui.netlify.app/ Bundle size report |
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.
@mnajdova What's your thought about 0.0.1
without the alpha? I don't think we need alpha if we follow Semver.
@brijeshb42 Have you tried pnpm publish --filter="@mui/zero-*" --tag latest --dry-run
?
476f0f9
to
f13e627
Compare
Yes. This works. I'll also test actual publishing to csb later and fix if any bugs are found. |
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.
@brijeshb42 I think after removing alpha from the version, this should be ready to merge.
f13e627
to
c4220e8
Compare
eaab7de
to
4a77d9d
Compare
Includes * Updated metadata in package.json * Added License banner to the output files * Copy LICENSE file to the packages * Add @emotion/styled @emotion/react as direct dep of zero-runtime, event though both aren't used anywhere, but zero-runtime depends on system which in turn requires these to be present. This will create an issue with standalone zero-runtime installation otherwise.
4a77d9d
to
de67a7f
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.
🎉🎉🎉🎉
Includes