Skip to content

Theme preview workflow: use external github action#232

Closed
vcanales wants to merge 2 commits intotrunkfrom
use-external-action-for-theme-preview
Closed

Theme preview workflow: use external github action#232
vcanales wants to merge 2 commits intotrunkfrom
use-external-action-for-theme-preview

Conversation

@vcanales
Copy link
Copy Markdown
Member

This proposes using an external copy of the preview themes workflow, which will allow us to run it on pull_request_target safely, since the code for the workflow itself lives outside of the repo — this means that the workflow will be able to comment on PRs created by non-members of the WordPress GitHub org.

The code is currently under my GitHub account, but I'll be moving to change ownership to the WordPress org eventually.

@vcanales vcanales added the enhancement New feature or request label Aug 16, 2024
@vcanales vcanales requested a review from MaggieCabrera August 16, 2024 05:24
@MaggieCabrera
Copy link
Copy Markdown
Collaborator

@mikachan did we implement this on the a8c themes? I can't remember if it had security issues

@mikachan
Copy link
Copy Markdown
Member

Yes, I think using the external copy worked around the security issues and we were able to use pull_request_target: Automattic/themes#8011.

@MaggieCabrera
Copy link
Copy Markdown
Collaborator

Awesome, I'll review this asap then. And it would be great to bring the theme check action too to this repo

@vcanales
Copy link
Copy Markdown
Member Author

Keep in mind that the action has been moved to https://github.com/WordPress/action-wp-playground-pr-preview

@mikachan
Copy link
Copy Markdown
Member

Thanks @vcanales! I went ahead and updated the action ref: 892208b.

@vcanales vcanales closed this Nov 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants