Skip to content

Add zaerl to GitHub workflows actors - #2041

Merged
adamziel merged 4 commits into
trunkfrom
add/workflow-zaerl
Dec 2, 2024
Merged

Add zaerl to GitHub workflows actors#2041
adamziel merged 4 commits into
trunkfrom
add/workflow-zaerl

Conversation

@zaerl

@zaerl zaerl commented Dec 2, 2024

Copy link
Copy Markdown
Collaborator

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.

@akirk

akirk commented Dec 2, 2024

Copy link
Copy Markdown
Member

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

zaerl commented Dec 2, 2024

Copy link
Copy Markdown
Collaborator Author

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
@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