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] Automate cherry-pick of PRs from next -> master #41741

Merged
merged 1 commit into from Apr 2, 2024

Conversation

aarongarciah
Copy link
Member

@aarongarciah aarongarciah commented Apr 2, 2024

As @oliviertassinari suggested, this PR adds a GitHub Action to automate cherry-picking PRs from next to master (see Olivier's comment to see how it works). The code is an exact copy from MUI X (thanks @MBilalShafi !).

cc @danilo-leal

@aarongarciah aarongarciah added the needs cherry-pick The PR should be cherry-picked to master after merge label Apr 2, 2024
@mui-bot
Copy link

mui-bot commented Apr 2, 2024

Netlify deploy preview

https://deploy-preview-41741--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 00643e8

if: ${{ contains(github.event.pull_request.labels.*.name, 'needs cherry-pick') && github.event.pull_request.merged == true }}
steps:
- name: Checkout
uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.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.

@mnajdova is there a reason to use the commit hash notation instead of using the version tag?

Suggested change
uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
uses: actions/checkout@v4.1.1

Copy link
Member

Choose a reason for hiding this comment

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

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.

Looks good, it's a nice contribution! I will update our guide to include adding this label if the PR needs to be cherry-picked.

@aarongarciah aarongarciah merged commit d568c56 into mui:next Apr 2, 2024
20 checks passed
aarongarciah added a commit to aarongarciah/material-ui that referenced this pull request Apr 2, 2024
@danilo-leal
Copy link
Contributor

Awesome first PR! 🤝

@oliviertassinari
Copy link
Member

Nice 👌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs cherry-pick The PR should be cherry-picked to master after merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants