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

Add zaerl to GitHub workflows actors #2041

Merged
merged 4 commits into from
Dec 2, 2024
Merged

Add zaerl to GitHub workflows actors #2041

merged 4 commits into from
Dec 2, 2024

Conversation

zaerl
Copy link
Collaborator

@zaerl zaerl commented Dec 2, 2024

Motivation for the change, related issues

I need to run this workflow, as well of the future ones.

Implementation details

Add myself to the if instructions of the GitHub workflows.

Testing Instructions (or ideally a Blueprint)

Nothing testable. Once merged, I will be able to run https://github.com/WordPress/wordpress-playground/actions/workflows/publish-npm-packages.yml and others.

@zaerl zaerl requested a review from a team December 2, 2024 08:09
@zaerl zaerl self-assigned this Dec 2, 2024
@zaerl zaerl changed the title Add zaerl to Gituhub workflows actors Add zaerl to GitHub workflows actors Dec 2, 2024
@akirk
Copy link
Member

akirk commented Dec 2, 2024

In the comments of the yml files the usernames are also mentioned which is now outdated. Could you please update the comments, too? Maybe to something more evergreen?

@zaerl
Copy link
Collaborator Author

zaerl commented Dec 2, 2024

In the comments of the yml files the usernames are also mentioned which is now outdated. Could you please update the comments, too? Maybe to something more evergreen?

Sure, let me know what you think about the new comments. A "part of the maintainers' group of users" check is probably usable in the GitHub workflow. That will be evergreen. But I think it's okay this way. We can optimize it later.

@adamziel adamziel merged commit ff1e3ed into trunk Dec 2, 2024
10 checks passed
@adamziel adamziel deleted the add/workflow-zaerl branch December 2, 2024 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants