-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[infra] Use CI action from mui-public #18709
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
Conversation
Deploy preview: https://deploy-preview-18709--material-ui-x.netlify.app/ Bundle size reportTotal Size Change: 0B(0.00%) - Total Gzip Change: 0B(0.00%) Show details for 100 more bundles (22 more not shown)@mui/x-charts parsed: 0B(0.00%) gzip: 0B(0.00%) |
9c85a5d
to
890bc9d
Compare
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
b5a021a
to
33ff4e9
Compare
85ee3f5
to
3935354
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.
I'm a bit on the fence on whether we should just use the composite everywhere. It could prevent maintainers from accidentally double building in the fire. For now this is probably fine though 👍
Skipping argos check since its not related. |
Co-authored-by: Brijesh Bittu <brijesh42@gmail.com>
Companion PRs for full fix -