Skip to content

Conversation

@dkotter
Copy link
Collaborator

@dkotter dkotter commented Dec 11, 2025

What?

Update our Build Plugin Zip and Pull Request Comments workflows to use the new WordPress/action-wp-playground-pr-preview Action.

Why?

We've had a few issues with our current setup (see #65 for latest work there) and we've been asked to try using the WordPress/action-wp-playground-pr-preview Action to see if that makes things work.

How?

Follows the instructions in the WordPress/action-wp-playground-pr-preview repo: https://github.com/WordPress/action-wp-playground-pr-preview/?tab=readme-ov-file#advanced-testing-built-ci-artifacts

  • Have our Build Plugin Zip workflow fire when a PR is opened
  • Update our Pull Request Comments workflow to fire after the Build Plugin Zip workflow has finished
  • Update this workflow to use the new WordPress/action-wp-playground-pr-preview Action

Testing Instructions

Ensure a Playground test link gets added to this PR description and it works as expected

@dkotter dkotter self-assigned this Dec 11, 2025
@github-actions
Copy link

github-actions bot commented Dec 11, 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.

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: dkotter <[email protected]>
Co-authored-by: jeffpaul <[email protected]>
Co-authored-by: adamziel <[email protected]>

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

@jeffpaul jeffpaul added this to the 0.2.0 milestone Dec 13, 2025
Comment on lines +6 to +10
on:
workflow_run:
workflows: ["Build Plugin Zip"]
types:
- completed
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For some reason this doesn't seem to get triggered, at least in this PR.

Copy link
Contributor

@adamziel adamziel Dec 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jeffpaul @dkotter This is probably the crux:

This event will only trigger a workflow run if the workflow file exists on the default branch.

https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflow_run

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, thanks for that. I think then the only way to fully test this will be to merge it in and 🤞🏻

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merged, now crossing fingers...

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Workflow is failing with a permission error. Opened #150 to try and fix that but still gives the same error 😢

@jeffpaul jeffpaul merged commit 0e9c94b into develop Dec 18, 2025
25 of 26 checks passed
@jeffpaul jeffpaul deleted the try/playground-action branch December 18, 2025 23:21
@github-project-automation github-project-automation bot moved this from In progress to Done in WordPress AI Planning & Roadmap Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

4 participants