Skip to content

Conversation

adamziel
Copy link
Collaborator

@adamziel adamziel commented Oct 1, 2025

Motivation for the change, related issues

#2694 is trying to attach to a local remote (127.0.0.1) instead of https://playground.wordpress.net/remote.html. This PR makes it use a remote appropriate for a given environment by leveraging the same website-config virtual vite module as the @wp-playground/website package

@adamziel adamziel added [Type] Bug An existing feature does not function as intended [Aspect] Website [Package][@wp-playground] Website labels Oct 1, 2025
@adamziel adamziel changed the title [PHP Playground] Attach client to http://playground.wordpress.net/remote.html [PHP Playground] Attach client to https://playground.wordpress.net/remote.html Oct 1, 2025
);
logger.warn(
`Skipping ${wpPath} because something exists at the target path.`
`Cannot unzip WordPress files at ${target}: ${wpPath} already exists.`
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is technically unrelated and got coupled with this PR by accident. Still, it's a useful and safe change in a logged string so I'll keep it here (shocked gasps).

@adamziel adamziel merged commit bf12d78 into trunk Oct 1, 2025
26 checks passed
@adamziel adamziel deleted the beta-php-playgorund-source-remote-from-url branch October 1, 2025 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Aspect] Website [Package][@wp-playground] Website [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant