Skip to content

Playground preview: simplify artifact handoff to the publisher - #1471

Merged
obenland merged 1 commit into
trunkfrom
update/playground-preview-publish
Sep 9, 2026
Merged

Playground preview: simplify artifact handoff to the publisher#1471
obenland merged 1 commit into
trunkfrom
update/playground-preview-publish

Conversation

@obenland

@obenland obenland commented Sep 9, 2026

Copy link
Copy Markdown
Member

Follow-up to #1454.

  • Resolve the pull request from the workflow_run payload (head repository, branch and SHA) instead of a metadata artifact, and drop the metadata steps from the build workflow.
  • Have the publisher download the plugin zip itself, check that it is a plain, valid zip file, and pass it to the expose action through the current run instead of the build run.
  • Pin the actions used by the publish workflow to commits.
  • Stop persisting checkout credentials in the build workflow and remove the stale trigger name.

🤖 Generated with Claude Code

Copilot AI lite review requested due to automatic review settings September 9, 2026 17:55

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

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: obenland <obenland@git.wordpress.org>

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

@obenland
obenland force-pushed the update/playground-preview-publish branch from 36a5d1c to 8d872b1 Compare September 9, 2026 17:57
Resolve the pull request from the workflow_run payload instead of a
metadata artifact, so the build workflow only produces the plugin zip.
Have the publisher download that zip itself, check that it is a plain,
valid zip file, and pass it to the expose action through the current
run rather than the build run. Pin the publish workflow's actions to
commits, stop persisting checkout credentials in the build, and drop
the stale trigger name.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@obenland
obenland force-pushed the update/playground-preview-publish branch from 8d872b1 to 88f964e Compare September 9, 2026 18:01
@obenland
obenland merged commit d4e6698 into trunk Sep 9, 2026
3 checks passed
@obenland
obenland deleted the update/playground-preview-publish branch September 9, 2026 18:05
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.

2 participants