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.

Sign up for GitHub

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

Jump to bottom

[core] Clarify release version bump strategy #15219

Merged
merged 4 commits into from
Nov 12, 2024
Merged

[core] Clarify release version bump strategy #15219

merged 4 commits into from
Nov 12, 2024

Conversation

cherniavskii
Copy link
Member

@cherniavskii cherniavskii commented Oct 31, 2024 •
edited
Loading

@cherniavskii cherniavskii added the core Infrastructure work going on behind the scenes label Oct 31, 2024
@cherniavskii cherniavskii mentioned this pull request Oct 31, 2024
Merged
@cherniavskii cherniavskii requested review from a team October 31, 2024 15:44
@mui-bot
Copy link

mui-bot commented Oct 31, 2024 •
edited
Loading

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

Generated by 🚫 dangerJS against dad922c

LukasTy
LukasTy approved these changes Oct 31, 2024
View reviewed changes
scripts/README.md Outdated Show resolved Hide resolved
scripts/README.md Outdated Show resolved Hide resolved
flaviendelangle
flaviendelangle reviewed Oct 31, 2024
View reviewed changes
Copy link
Member

@flaviendelangle flaviendelangle left a comment

Choose a reason for hiding this comment

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

I'll bring the topic on the eXplore/xCharts weekly meeting on Monday to have an approval from our side 👍

@cherniavskii @LukasTy
Apply suggestions from code review
e097b18 
Co-authored-by: Lukas Tyla <[email protected]>
Signed-off-by: Andrew Cherniavskii <[email protected]>
KenanYusuf
KenanYusuf reviewed Oct 31, 2024
View reviewed changes
scripts/README.md Outdated Show resolved Hide resolved
alexfauquette
alexfauquette reviewed Oct 31, 2024
View reviewed changes
scripts/README.md Outdated Show resolved Hide resolved
@cherniavskii @alexfauquette @KenanYusuf
Apply suggestions from code review
dad922c 
Co-authored-by: Alexandre Fauquette <[email protected]>
Co-authored-by: Kenan Yusuf <[email protected]>
Signed-off-by: Andrew Cherniavskii <[email protected]>
@cherniavskii cherniavskii added the needs cherry-pick The PR should be cherry-picked to master after merge label Nov 12, 2024
@cherniavskii cherniavskii merged commit 0c54c5b into mui:master Nov 12, 2024
20 checks passed
@cherniavskii cherniavskii deleted the clarify-version-bump branch November 12, 2024 15:05
cherniavskii added a commit to cherniavskii/mui-x that referenced this pull request Nov 21, 2024
@cherniavskii @LukasTy
@alexfauquette @KenanYusuf
[core] Clarify release version bump strategy (mui#15219)
b39a310 
Co-authored-by: Lukas Tyla <[email protected]>
Co-authored-by: Alexandre Fauquette <[email protected]>
Co-authored-by: Kenan Yusuf <[email protected]>
@cherniavskii cherniavskii mentioned this pull request Nov 21, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LukasTy LukasTy LukasTy approved these changes

@noraleonte noraleonte noraleonte approved these changes

@alexfauquette alexfauquette alexfauquette approved these changes

@arthurbalduini arthurbalduini arthurbalduini approved these changes

@michelengelen michelengelen michelengelen approved these changes

@JCQuintas JCQuintas JCQuintas approved these changes

@KenanYusuf KenanYusuf KenanYusuf approved these changes

@arminmeh arminmeh arminmeh approved these changes

@flaviendelangle flaviendelangle flaviendelangle approved these changes

Assignees
No one assigned
Labels
core Infrastructure work going on behind the scenes needs cherry-pick The PR should be cherry-picked to master after merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.