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.0.0-beta.6 #12375

Merged
merged 8 commits into from
Mar 8, 2024
Merged

v7.0.0-beta.6 #12375

merged 8 commits into from
Mar 8, 2024

Conversation

flaviendelangle
Copy link
Member

@flaviendelangle flaviendelangle commented Mar 8, 2024

Nothing to release on master this week

Which packages are released

We had a chat last week with @cherniavskii, @joserodolfofreitas and @DanailH and it seems that we don't all agree about which packages should be released based on the changes done during the week and with which version.
So we decided that, until we reached to an agreement, we would just follow what Lerna suggests.

@@ -246,7 +246,7 @@ Same changes as in \`@mui/x-data-grid-pro@__VERSION__\`${
dataGridPremiumCommits.length > 0 ? ', plus:\n' : '.'
}
${logChangelogSection(dataGridPremiumCommits)}${dataGridPremiumCommits.length > 0 ? '\n' : ''}
### Date Pickers
### Date and Time Pickers
Copy link
Member Author

Choose a reason for hiding this comment

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

We try to use the same wording everywhere, I'm surprised that this occurrence was not spotted before

Copy link
Member

Choose a reason for hiding this comment

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

🙈

@mui-bot
Copy link

mui-bot commented Mar 8, 2024

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

Generated by 🚫 dangerJS against 0b65bbe

@@ -1,6 +1,6 @@
{
"name": "@mui/x-license",
"version": "7.0.0-beta.2",
"version": "7.0.0-beta.6",
Copy link
Member

Choose a reason for hiding this comment

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

I would have voted for consistency with previous approaches before a consensus is reached, but if you decided otherwise, so be it. 👌

scripts/releaseChangelog.mjs Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
@bharatkashyap bharatkashyap removed their request for review March 8, 2024 09:41
flaviendelangle and others added 4 commits March 8, 2024 14:34
Co-authored-by: Lukas <[email protected]>
Signed-off-by: Flavien DELANGLE <[email protected]>
Co-authored-by: Lukas <[email protected]>
Signed-off-by: Flavien DELANGLE <[email protected]>
@flaviendelangle flaviendelangle requested a review from LukasTy March 8, 2024 14:08
Copy link
Member

@michelengelen michelengelen left a comment

Choose a reason for hiding this comment

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

one nit ... LGTM :shipit:

CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Michel Engelen <[email protected]>
Signed-off-by: Flavien DELANGLE <[email protected]>
@flaviendelangle flaviendelangle merged commit f96c319 into mui:next Mar 8, 2024
17 checks passed
@flaviendelangle flaviendelangle deleted the v7.0.0-beta.6 branch March 8, 2024 15:35
thomasmoon pushed a commit to thomasmoon/mui-x that referenced this pull request Sep 9, 2024
Signed-off-by: Flavien DELANGLE <[email protected]>
Co-authored-by: Lukas <[email protected]>
Co-authored-by: Michel Engelen <[email protected]>
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.

5 participants