Skip to content

feat: Bump minimum PHP version to 8.2#151

Merged
justlevine merged 13 commits into
developfrom
feat/php8.2
Mar 10, 2026
Merged

feat: Bump minimum PHP version to 8.2#151
justlevine merged 13 commits into
developfrom
feat/php8.2

Conversation

@justlevine
Copy link
Copy Markdown
Collaborator

@justlevine justlevine commented Mar 9, 2026

Description

Bumps the minimum PHP version to 8.2

Important

This PR is based on #146 which should be merged first.

Technical Details

Precursor to implementing PHPUnit tests and moving to v4.0+ of the Algolia library.

Checklist

Screenshots

To-do

Fixes/Covers issue

Fixes #

Open WordPress Playground Preview

@justlevine justlevine changed the base branch from test/skeleton-migration to develop March 9, 2026 23:55
@justlevine justlevine changed the base branch from develop to test/skeleton-migration March 10, 2026 00:13
@justlevine justlevine requested a review from Copilot March 10, 2026 00:13
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the plugin/tooling metadata to reflect a new minimum supported PHP version (8.2), aligning CI and static analysis with that baseline as a precursor to PHPUnit adoption and future dependency upgrades.

Changes:

  • Bump “Requires PHP” to 8.2 across plugin readmes and the plugin header.
  • Update dev tooling baselines (PHPStan min version, PHPCS compatibility testVersion, Composer platform PHP).
  • Update PHPUnit configuration and CI matrix to run only on PHP 8.2+.

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
readme.txt Updates WordPress plugin readme “Requires PHP” to 8.2.
README.md Updates documentation requirements table to PHP >= 8.2.
onesearch.php Updates plugin header “Requires PHP” to 8.2.
composer.json Updates Composer platform PHP to 8.2 for dependency resolution.
composer.lock Regenerates lockfile for the updated platform/dependency set (incl. newer PHPUnit stack).
phpstan.neon.dist Raises PHPStan phpVersion.min to 80200 (PHP 8.2).
.phpcs.xml.dist Raises PHPCS PHPCompatibility testVersion baseline to 8.2-.
phpunit.xml.dist Updates PHPUnit config to the v11.5 schema and new settings/paths.
.github/workflows/ci.yml Removes PHP 8.0/8.1 from the PHPUnit job matrix.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread phpunit.xml.dist Outdated
Comment thread phpunit.xml.dist
Comment thread composer.json
justlevine and others added 2 commits March 10, 2026 02:42
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@justlevine justlevine requested a review from up1512001 March 10, 2026 01:36
Copy link
Copy Markdown
Member

@up1512001 up1512001 left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from test/skeleton-migration to develop March 10, 2026 14:40
@justlevine justlevine merged commit 944b536 into develop Mar 10, 2026
17 of 19 checks passed
@justlevine justlevine deleted the feat/php8.2 branch March 10, 2026 15:39
This was referenced Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants