-
-
Notifications
You must be signed in to change notification settings - Fork 641
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
feat(ci): 🔖 add appropriate labels to PRs with failed checks #2785
Conversation
Add steps to workflows to manage labels based on workflow results. * **Color Check Workflow**: - Add permission to write pull requests. - Add step to manage `invalid-colors` label based on color check result. * **Build Workflow**: - Add permission to write pull requests. - Add step to manage `invalid-build` label based on build result. * **PR Title Workflow**: - Add step to manage `invalid-title` label based on PR title check result. --- For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/material-extensions/vscode-material-icon-theme?shareId=XXXX-XXXX-XXXX-XXXX).
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.
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
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.
I'm wondering if it would be worth checking if we can make use of reusable workflows:
https://docs.github.com/en/actions/sharing-automations/reusing-workflows
Co-authored-by: Philipp Kief <[email protected]> Signed-off-by: Okinea Dev <[email protected]>
Co-authored-by: Philipp Kief <[email protected]> Signed-off-by: Okinea Dev <[email protected]>
Co-authored-by: Philipp Kief <[email protected]> Signed-off-by: Okinea Dev <[email protected]>
@okineadev btw, the label has been added again :D |
dont merge |
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.
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
Files not reviewed (1)
- .github/actions/update-status-label.sh: Language not supported
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.
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
Files not reviewed (1)
- .github/actions/update-status-label.sh: Language not supported
This comment was marked as off-topic.
This comment was marked as off-topic.
After you merge this PR, I will create a new PR with the updated label format with emoji ( |
Merge SuccessfulThanks for your contribution! 🎉 The changes will be part of the upcoming update on the Marketplace. |
…l-extensions#2785) Signed-off-by: Okinea Dev <[email protected]> Co-authored-by: Philipp Kief <[email protected]>
Signed-off-by: Okinea Dev <[email protected]> Co-authored-by: Philipp Kief <[email protected]>
Add steps to workflows to manage labels based on workflow results.
This will provide a convenient understanding of the problem with PR
Color Check Workflow:
invalid-colors
label based on color check result.Build Workflow:
invalid-build
label based on build result.PR Title Workflow:
invalid-title
label based on PR title check result.For more details, open the Copilot Workspace session.