Skip to content

Commit 5e30b2e

Browse files
pattonwebzclaude
andcommitted
Trim the Playground step comment
Most of it explained steps that no longer exist, or restated what the step's own `if`/`with` already says. Keeps only the two things reading the step won't tell you. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_019LoLgW7oFjBPxGea9kiZJ2
1 parent 1075d5a commit 5e30b2e

1 file changed

Lines changed: 2 additions & 14 deletions

File tree

.github/workflows/build-plugin-with-ref.yml

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -255,20 +255,8 @@ jobs:
255255
path: ${{ env.REF_ZIP_FOLDER }}
256256
if-no-files-found: error
257257

258-
# Playground links are built by pattonwebz/wordpress-playground-action, which
259-
# resolves the uploaded artifact to a public nightly.link URL and assembles the
260-
# blueprint.
261-
#
262-
# Only the primary build gets a link. Not run on `release`: releases carry the
263-
# real zip as a release asset, and nightly.link only serves Actions artifacts.
264-
# The ref (woocommerce) build gets no link either - `need_ref_build` is never
265-
# true for a pull_request, so a ref link could only ever have surfaced in a
266-
# manual-dispatch job log, never in the PR comment.
267-
#
268-
# plugin-slug is set explicitly rather than inferred: the action strips a
269-
# version/hash suffix (accessibility-checker-1.47.0-123-abc1234 -> accessibility-checker)
270-
# but would leave the whole "…-1.47.0-ref-woocommerce-abc1234" shape intact,
271-
# so pinning it keeps the activation path correct regardless of artifact naming.
258+
# Skipped on release: nightly.link serves Actions artifacts, not release assets.
259+
# plugin-slug is pinned so the activation path never depends on slug inference.
272260
- name: Playground link (primary build)
273261
id: playground_primary
274262
if: steps.setref.outputs.skip_primary != 'true' && github.event_name != 'release'

0 commit comments

Comments
 (0)