-
-
Notifications
You must be signed in to change notification settings - Fork 643
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 labels to PRs and use reusable workflows #2829
Conversation
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 5 out of 5 changed files in this pull request and generated no comments.
Needed for material-extensions/vscode-material-icon-theme#2829 because it uses a separate workflow for adding labels Signed-off-by: Okinea Dev <[email protected]>
|
When this is done I will be able to test |
* Removed `contents: read` permission from the `pr-closed.yml` workflow.
Signed-off-by: Okinea Dev <[email protected]>
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.
At the moment there's no capacity to elaborate on workflow files. We already had some additional effort in the past to fix failed workflow files. It makes the maintenance of the project more cumbersome and I want to avoid that. That's why the plans for adding labels automatically are stopped now.
Description
This approach uses an isolated environment for validations and a privileged environment for tagging, which ensures security
Contribution Guidelines