Repository navigation
Env: Remove non-functional WP_ENV_MULTISITE config - #72567
Conversation
e2f4cd6 to
99bec1b
Compare
|
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 If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
|
Size Change: 0 B Total Size: 3 MB ℹ️ View Unchanged
|
99bec1b to
c21eb28
Compare
c21eb28 to
5510117
Compare
|
Flaky tests detected in 5510117. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/21580668217
|
|
Maybe this variable is not needed anymore if we offer #75087 |
|
I understand your concern, but this PR fixes a feature that is documented but doesn't actually work: gutenberg/packages/env/README.md Lines 786 to 788 in 4d8d180 Another option is to remove any mention of the non-working |
|
So that PR has been merged a year ago, and it's not working since then right? This makes me think that it might not be worth keeping this feature/variable around. WDYT? |
WP_ENV_MULTISITE environment variable override not working correctlyWP_ENV_MULTISITE config
Yes 😅 Let's remove this config. |
* Env: Fix `WP_ENV_MULTISITE` environment variable override not working correctly * Remove non-functional WP_ENV_MULTISITE config Co-authored-by: t-hamano <wildworks@git.wordpress.org> Co-authored-by: youknowriad <youknowriad@git.wordpress.org>
Follow-up #68792
What?
I noticed that the
WP_ENV_MULTISITEenvironment variable override not working correctly.Why?
The necessary code was missing from the
getEnvironmentVarOverridesfunction.How?
Multisite is enabled when set to1ortrue, as follows:WP_ENV_MULTISITE=1 npm run wp-env startWP_ENV_MULTISITE="1" npm run wp-env startWP_ENV_MULTISITE=true npm run wp-env startWP_ENV_MULTISITE="true" npm run wp-env startAfter a discussion, we decided to remove the non-functional
WP_ENV_MULTISITEconfig. Themultisitefield inwp-env.jsonworks as before.Testing Instructions
Run either of the above commands, and confirm that the multisite environment is launched.Confirm that the
multisitefield inwp-env.jsonworks as before:{ "multisite": true }