Raise minimum PHP to 8.2 and expand CI matrix (XWPENG-46) - #1978
Open
shadyvb wants to merge 2 commits into
Open
Conversation
shadyvb
marked this pull request as draft
September 1, 2026 15:16
shadyvb
force-pushed
the
ticket/XWPENG-46-php-minimum
branch
from
September 1, 2026 15:16
143e713 to
0067067
Compare
shadyvb
marked this pull request as ready for review
September 1, 2026 15:22
…rced end-to-end (XWPENG-46). Replace shutdown echo with capability-gated admin notices, bump release to 5.0.0, and quiet PHPUnit multisite noise from headers and Yoast indexables.
shadyvb
force-pushed
the
ticket/XWPENG-46-php-minimum
branch
from
September 1, 2026 15:22
de6396e to
0956d64
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
composer.json^8.2,Requires PHP: 8.2, PHPCStestVersion8.2-). Sites below 8.2 get a capability-gatedadmin_notices/network_admin_noticesmessage; Stream does not bootstrap (no fatal, no shutdown echo into REST/AJAX/feeds/CLI).WP_ENV_PHP_VERSION, andcomposer check-platform-reqsafter install. Deploy workflows use PHP 8.2.WP_TEST_ACTIVATED_PLUGINSapplied viasite_option_active_sitewide_pluginsmap (fixes Jetpack path noise); testcase silences post-bootstrap header/cookie warnings and Yoast indexable watchers on factory blogs; PHPCS/doc tweaks andAdmin_Teststub fix; removed obsolete PHP 5.3.6 backtrace guard inLog. Host unit coverage inFail_Php_Version_Unit_Test(hook registration + capability-gated notice output).Stack context
Stacks on #1977 (
ticket/XWPENG-45-wp-env) → part of GitHub stack #1976 (develop ← … ← this PR).Test plan
composer check-platform-reqson PHP 8.2+composer test-unitnpm run lintnpm run test(wp-env, single-site + multisite)npm run test-e2e(at least one matrix leg in CI)Fail_Php_Version_Unit_Test(admin notice hooks +activate_pluginsgate)Branch protection migration (maintainers)
CI job display names changed from static
Lint and Test/E2Eto matrix names. Update required status checks ondevelop/masterbefore or right after merge:Lint and Test (PHP 8.2)Lint and Test (PHP 8.3)Lint and Test (PHP 8.4)E2E (PHP 8.2)E2E (PHP 8.3)E2E (PHP 8.4)Remove obsolete
Lint and TestandE2Eentries. Keeping only the old names blocks every PR; removing them without adding the six allows merges without CI.Review artifacts
Local review notes under
.ai/reviews/(gitignored): Stage 4 applied admin-notice fix (S2-H1), version/changelog 5.0.0 (S2-H2), platform-reqs in CI (S2-M1), and documented branch-protection migration (S2-H3).