-
-
Notifications
You must be signed in to change notification settings - Fork 32.7k
[code-infra] Publish from CI #46851
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
base: master
Are you sure you want to change the base?
[code-infra] Publish from CI #46851
Conversation
Netlify deploy previewhttps://deploy-preview-46851--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.
I was wonderying why the file publish.yml is not npm-publish.yml to match the environment name (npm-publish). It couldn't make sense of why it's different.
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.
No specific reason, I included this after building the whole thing and I think I wanted it to stand out a bit more in the barrage of environments that exist in mui/mui-public
.
It'll requiring a manual update of every npm package to change this. Not sure it adds enough value to spend the time on this?
Add CI publishing. This brings provenance and some automation around github releases.
@mui/material-ui Ping me when you know next release happens, so I can make sure I'm around to assist in case there are still misconfigurations