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

[core] Remove --exact from release:version #28840

Merged
merged 4 commits into from
Oct 6, 2021

Conversation

siriwatknp
Copy link
Member

@siriwatknp siriwatknp commented Oct 5, 2021

This will help us not to manually update the version with ^ when doing release. @michaldudak please try it out in this release and see if there is any error with this change.

@siriwatknp siriwatknp requested review from eps1lon, mnajdova and a team and removed request for mnajdova and eps1lon October 5, 2021 08:44
@mui-pr-bot
Copy link

mui-pr-bot commented Oct 5, 2021

No bundle size changes

Generated by 🚫 dangerJS against 6ab418e

@eps1lon
Copy link
Member

eps1lon commented Oct 5, 2021

Sounds like you didn't actually test this?

@siriwatknp
Copy link
Member Author

Sounds like you didn't actually test this?

Sorry, I should have mentioned that it is tested locally (carets are applied).

@mnajdova
Copy link
Member

mnajdova commented Oct 5, 2021

@siriwatknp does this takes into account the alpha versions? For example, not adding carets on those versions? If not, it would be best if we would document this in the scripts/README.md

@siriwatknp
Copy link
Member Author

@siriwatknp does this takes into account the alpha versions? For example, not adding carets on those versions? If not, it would be best if we would document this in the scripts/README.md

No, caret also added to the packages with alpha version. I will add some note in README to remove the caret for alpha versions.

@siriwatknp siriwatknp force-pushed the release-without-exact branch from c9edc1e to 664a830 Compare October 6, 2021 02:52
scripts/README.md Outdated Show resolved Hide resolved
Copy link
Member

@mnajdova mnajdova left a comment

Choose a reason for hiding this comment

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

I've pushed a commit for a typo. Looks good to me.

scripts/README.md Outdated Show resolved Hide resolved
@siriwatknp siriwatknp merged commit 208a718 into mui:master Oct 6, 2021
@zannager zannager added the core Infrastructure work going on behind the scenes label Jan 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Infrastructure work going on behind the scenes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants