Skip to content

Sync the test workflow with the example.#201

Open
desrosj wants to merge 4 commits intotrunkfrom
run-action-for-label
Open

Sync the test workflow with the example.#201
desrosj wants to merge 4 commits intotrunkfrom
run-action-for-label

Conversation

@desrosj
Copy link
Member

@desrosj desrosj commented Sep 2, 2025

What?

This updates the workflow that runs the Props Bot action within this repository to better match the example.

Why?

This ensures that the example in the repository is tested, and accounts for any discovered edge cases.

Currently, there's no support for the pull_request event labelled type, so the label has no affect. So there's no way to manually generate a list of props. This is important when a pull request originates from a fork.

How?

The ./example.props-bot.yml file has been copied into the .github/workflows/test-action.yml one.

A few changes have been made, each being accompanied by an inline comment that notes and explains the change for future syncs.

This ensures that the example in the repository is tested, and accounts for any discovered edge cases.

Currently, there's no support for the `pull_request` event `labelled` type, so the label has no affect.
This makes a few adjustments to the suggested workflow file in the example to account for running within the action repository itself.

All differences are noted with inline comments.
@desrosj desrosj self-assigned this Sep 2, 2025
@github-actions
Copy link

github-actions bot commented Sep 2, 2025

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core SVN

Core Committers: Use this line as a base for the props when committing in SVN:

Props desrosj.

GitHub Merge commits

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: desrosj <desrosj@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@desrosj desrosj requested a review from aaronjorbin September 2, 2025 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant