-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
chore(github): add workflow and action for automating review labels #5649
chore(github): add workflow and action for automating review labels #5649
Conversation
Deploy preview for carbon-elements ready! Built with commit a56c51e |
Deploy preview for carbon-components-react ready! Built with commit a56c51e https://deploy-preview-5649--carbon-components-react.netlify.com |
Deploy preview for carbon-elements ready! Built with commit c63d94b |
Deploy preview for carbon-components-react ready! Built with commit c63d94b https://deploy-preview-5649--carbon-components-react.netlify.com |
Super cool! Quick question, anyone who opts in to the automatic "ready to merge" label will get it after two reviews regardless of if the review is from dev or design? |
@abbeyhrt yup! |
…om:joshblack/carbon into chore/add-review-label-workflow-and-action
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
Fun little side-experiment with adding actions for automating our workflow. This one, in particular, will coordinate adding/removing the "one more review" and "ready to merge" labels. The "ready to merge" labels are automatically applied for the users specified in the GitHub Workflow.
I've been experimenting with this workflow/action over in: joshblack/actions-playground#1
In addition, this PR adds in a top-level "actions" folder that is treated as part of our workspace in Yarn. This folder contains custom actions for the project. Actions are authored as Node.js files and executed using Doctor in a GitHub Workflow. As a result, there are some updates to the CLI package to generalize package paths outside of the
packages
directory.Changelog
New
Changed
Removed