chore: require PHP 8.2#1410
Conversation
|
Warning Review limit reached
More reviews will be available in 22 minutes and 32 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more credits in the billing tab to continue. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. 📝 WalkthroughWalkthroughThis PR raises the minimum required PHP version for the Ultimate Multisite plugin from 7.4.x to 8.2. The change is applied consistently across runtime requirement checks, Composer dependency constraints, CI/CD test matrices, and public documentation. ChangesPHP 8.2 Minimum Version Requirement
🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
🔨 Build Complete - Ready for Testing!📦 Download Build Artifact (Recommended)Download the zip build, upload to WordPress and test:
🌐 Test in WordPress Playground (Very Experimental)Click the link below to instantly test this PR in your browser - no installation needed! Login credentials: |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
composer.json (1)
33-83:⚠️ Potential issue | 🟡 MinorVerify PHP 8.2 dependency compatibility in
composer.json(with extension requirements)
- Dependencies resolve cleanly against the PHP 8.2 platform constraint (no PHP-version constraint conflicts).
- Composer resolution fails unless PHP extensions are enabled:
remotelyliving/php-dnsrequiresext-intlmpdf/mpdfrequiresext-gd- Ensure
intlandgdare present in CI/deployment environments runningcomposer install/update, otherwise the “PHP 8.2” change won’t be usable in practice.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@composer.json` around lines 33 - 83, composer.json declares PHP 8.2 but two required packages need PHP extensions that CI may not have; ensure ext-intl and ext-gd are present or handled: update CI/deployment images and composer config to require those extensions (ext-intl and ext-gd) so composer resolution succeeds with remotelyliving/php-dns and mpdf/mpdf, or add platform-check/extra notes and/or replace those packages with alternatives that don't need the extensions; specifically verify and add ext-intl and ext-gd to the environment used for composer install/update and, if desired, add them to composer.json require section to make the dependency explicit.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@inc/class-requirements.php`:
- Line 39: The repo still contains many textual and conditional references to
PHP 7.4; update them to reflect the new minimum PHP >= 8.2 by changing the class
property public static $php_version (inc/class-requirements.php) and
composer.json as the source of truth, then edit all remaining docs and comments
(README.md, CONTRIBUTING.md, AGENTS.md,
views/site-exporter/self-boot/readme.txt,
tests/WP_Ultimo/Site_Exporter/Self_Boot_Builder_Test.php) to say “PHP 8.2” or
“PHP 8.2.0+” and update any badges and phrases like “7.4”/“7.4.30” accordingly;
also update runtime checks that call version_compare(..., '7.1.0') or similar
(e.g., in mu-migration code) to compare against '8.2.0' (or use the $php_version
constant) so logic and messaging are consistent.
---
Outside diff comments:
In `@composer.json`:
- Around line 33-83: composer.json declares PHP 8.2 but two required packages
need PHP extensions that CI may not have; ensure ext-intl and ext-gd are present
or handled: update CI/deployment images and composer config to require those
extensions (ext-intl and ext-gd) so composer resolution succeeds with
remotelyliving/php-dns and mpdf/mpdf, or add platform-check/extra notes and/or
replace those packages with alternatives that don't need the extensions;
specifically verify and add ext-intl and ext-gd to the environment used for
composer install/update and, if desired, add them to composer.json require
section to make the dependency explicit.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: be9d61a0-1ce5-42ae-8eec-39febd572b14
⛔ Files ignored due to path filters (1)
composer.lockis excluded by!**/*.lock
📒 Files selected for processing (7)
.github/workflows/addon-integration-test.yml.github/workflows/e2e.yml.github/workflows/release.ymlcomposer.jsoninc/class-requirements.phpreadme.txtultimate-multisite.php
🔨 Build Complete - Ready for Testing!📦 Download Build Artifact (Recommended)Download the zip build, upload to WordPress and test:
🌐 Test in WordPress Playground (Very Experimental)Click the link below to instantly test this PR in your browser - no installation needed! Login credentials: |
|
CLAIM_RELEASED reason=worker_complete runner=superdav42 ts=2026-06-11T22:07:24Z aidevops_version=3.20.55 opencode_version=1.17.3 |
🔨 Build Complete - Ready for Testing!📦 Download Build Artifact (Recommended)Download the zip build, upload to WordPress and test:
🌐 Test in WordPress Playground (Very Experimental)Click the link below to instantly test this PR in your browser - no installation needed! Login credentials: |
|
CLAIM_RELEASED reason=worker_complete runner=superdav42 ts=2026-06-11T23:23:08Z aidevops_version=3.20.55 opencode_version=1.17.3 |
|
CLAIM_RELEASED reason=worker_complete runner=superdav42 ts=2026-06-12T00:44:53Z aidevops_version=3.20.55 opencode_version=1.17.3 |
🔨 Build Complete - Ready for Testing!📦 Download Build Artifact (Recommended)Download the zip build, upload to WordPress and test:
🌐 Test in WordPress Playground (Very Experimental)Click the link below to instantly test this PR in your browser - no installation needed! Login credentials: |
1 participant
Summary
Verification
php -l ultimate-multisite.php && php -l inc/class-requirements.phpvendor/bin/phpcs ultimate-multisite.php inc/class-requirements.phpcomposer validate --no-check-publishgit diff --checkNotes
composer validate --no-check-publishpasses with existing warnings about theversionfield and commit-pinned packages.aidevops.sh v3.20.55 plugin for OpenCode v1.17.3 with gpt-5.5 spent 58m and 303,609 tokens on this with the user in an interactive session.