v9.0.0-rc.0#21997
Conversation
|
Deploy preview: https://deploy-preview-21997--material-ui-x.netlify.app/ Bundle size report
|
Co-authored-by: Jose C Quintas Jr <juniorquintas@gmail.com> Co-authored-by: Siriwat K <siriwatkunaporn@gmail.com> Signed-off-by: Siriwat K <siriwatkunaporn@gmail.com>
| @@ -1,6 +1,6 @@ | |||
| { | |||
| "name": "@mui/x-scheduler-headless", | |||
| "version": "0.0.1-alpha.0", | |||
There was a problem hiding this comment.
We shouldn't change this version, it should be alpha - for all scheduler package.
There was a problem hiding this comment.
Btw, shouldn't this be 9.0.0-alpha.0? cc @flaviendelangle @rita-codes
There was a problem hiding this comment.
I'll wait for the confirmation
There was a problem hiding this comment.
I think it should be 9.0.0-alpha.0, yes, to be aligned with the monorepo versioning 👀
There was a problem hiding this comment.
Yep 👍
Same for the other 3 scheduler packages
There was a problem hiding this comment.
| "@mui/material": "^7.3.0 || ^9.0.0-beta.0 || ^9.0.0-beta.1 || ^9.0.0", |
We should do this for all packages, to make sure we can test this version once we release stable Material UI v9.
| { | ||
| "name": "@mui/x-internal-gestures", | ||
| "version": "0.4.1-alpha.0", | ||
| "version": "9.0.0-rc.0", |
There was a problem hiding this comment.
Is this bump expected @mui/charts?
| "@emotion/react": "^11.9.0", | ||
| "@emotion/styled": "^11.8.1", | ||
| "@mui/material": "^7.3.0 || ^9.0.0-beta.1", | ||
| "@mui/material": "^7.3.0 || ^9.0.0-beta.0 || ^9.0.0-beta.1 || ^9.0.0", |
There was a problem hiding this comment.
Wouldn't this be enough?
| "@mui/material": "^7.3.0 || ^9.0.0-beta.0 || ^9.0.0-beta.1 || ^9.0.0", | |
| "@mui/material": "^7.3.0 || ^9.0.0-beta.0 || ^9.0.0", |
There was a problem hiding this comment.
that's true, actually ^7.3.0 || ^9.0.0-beta.0 might be enough but let's be safe with your suggestion 🤣
✅ waiting for #21966
⌛️ waiting for #21666