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

feat(ci): 🔖 add labels to PRs and use reusable workflows #2829

Closed
wants to merge 4 commits into from

Conversation

okineadev
Copy link
Contributor

Description

This approach uses an isolated environment for validations and a privileged environment for tagging, which ensures security

Contribution Guidelines

@okineadev okineadev requested a review from Copilot January 20, 2025 07:18
@github-actions github-actions bot added the 🔄 workflows GitHub Actions label Jan 20, 2025

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.

okineadev added a commit to material-extensions/workflows that referenced this pull request Jan 20, 2025
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]>
@okineadev
Copy link
Contributor Author

⚠️ First we need to merge material-extensions/workflows#5

@okineadev okineadev marked this pull request as ready for review January 20, 2025 09:56
@okineadev
Copy link
Contributor Author

⚠️ First we need to merge material-extensions/workflows#5

When this is done I will be able to test pr-title.yml

@okineadev okineadev marked this pull request as draft January 21, 2025 05:44
* Removed `contents: read` permission from the `pr-closed.yml` workflow.
@okineadev okineadev marked this pull request as ready for review January 21, 2025 06:34
Copy link
Member

@PKief PKief left a 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.

@PKief PKief closed this Jan 26, 2025
@PKief PKief deleted the add-labels-for-prs-and-use-reusable-workflows branch January 26, 2025 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔄 workflows GitHub Actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants