Skip to content
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

v7.23.5 #16010

Merged
merged 5 commits into from
Dec 27, 2024
Merged

v7.23.5 #16010

merged 5 commits into from
Dec 27, 2024

Conversation

MBilalShafi
Copy link
Member

@MBilalShafi MBilalShafi commented Dec 27, 2024

Fix #16007

Skipping publishing of @mui/x-internals caused these changes mess up the codesandbox/stackblitz demos.

@MBilalShafi MBilalShafi added the release We are shipping :D label Dec 27, 2024
@MBilalShafi MBilalShafi requested a review from DanailH December 27, 2024 13:57
CHANGELOG.md Outdated Show resolved Hide resolved
Signed-off-by: Bilal Shafi <[email protected]>
CHANGELOG.md Outdated Show resolved Hide resolved
@@ -1,6 +1,6 @@
{
"name": "@mui/x-license",
"version": "7.23.2",
"version": "7.23.5",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did Lerna suggest bumping this package? 🤔
It doesn't look like it had any changes between 7.32.2 and now. 🤷

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also skipped it for the one I did.

Copy link
Member Author

@MBilalShafi MBilalShafi Dec 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lerna suggested nothing 😄
I had to use --force-publish to make it bump the packages.

I thought we usually bump x-grid-generator and x-license anyways, isn't it?
Or is it only with x-grid-generator 🤔

I'll revert x-license bump in that case.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm ok with bumping it but why not go to 7.23.3?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gotcha.
Did --force-publish suggest other product packages? 🤔

I'm ok with bumping it but why not go to 7.23.3?

Because we sync the package versions whenever we do a release. 😉

Copy link
Member Author

@MBilalShafi MBilalShafi Dec 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because we sync the package versions whenever we do a release. 😉

Do we? I also thought the same but later assumed it's not the case anymore and adjusted the versions to incrementing only one. 🤦

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did --force-publish suggest other product packages? 🤔

Yes, every package. I skipped the others manually.

Copy link
Member Author

@MBilalShafi MBilalShafi Dec 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because we sync the package versions whenever we do a release. 😉

Let's have a consensus on it and make it part of scripts/README.md

@mui/x Are we OK on syncing the package versions whenever we release?

👍 = YES 👎 = NO. Release should be done incrementing by 1

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR opened: #16012

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, so we only miss the version update for @mui/x-internals and @mui/x-license and we should be good to go 👍🏻

@DanailH DanailH requested a review from cherniavskii December 27, 2024 14:01
@mui-bot
Copy link

mui-bot commented Dec 27, 2024

Deploy preview: https://deploy-preview-16010--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 8c052ad

MBilalShafi and others added 2 commits December 27, 2024 19:15
Co-authored-by: Bilal Shafi <[email protected]>
Signed-off-by: Danail Hadjiatanasov <[email protected]>
This reverts commit aee81d7.
@MBilalShafi MBilalShafi merged commit b3091fe into mui:v7.x Dec 27, 2024
16 checks passed
@MBilalShafi MBilalShafi deleted the 7.23.5 branch December 27, 2024 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release We are shipping :D
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants