Skip to content

Update PHP version requirements in composer.json#16

Merged
donatj merged 3 commits into
masterfrom
donatj-patch-2
May 15, 2026
Merged

Update PHP version requirements in composer.json#16
donatj merged 3 commits into
masterfrom
donatj-patch-2

Conversation

@donatj
Copy link
Copy Markdown
Member

@donatj donatj commented May 15, 2026

Raises the floor to PHP 7.2 as it is the earliest supported by a working version of PHPUnit.

@donatj donatj marked this pull request as ready for review May 15, 2026 09:36
Copilot AI review requested due to automatic review settings May 15, 2026 09:36
Copy link
Copy Markdown

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

This pull request raises the minimum supported PHP version to 7.2 across the package metadata and CI configuration, aligning the library’s stated/runtime requirements with PHPUnit versions that can be used for testing.

Changes:

  • Updated the documented PHP requirement in README.md to PHP >= 7.2.0.
  • Updated composer.json to require PHP >= 7.2.0 and adjusted the PHPUnit dev dependency to versions compatible with that floor.
  • Trimmed the CI PHP matrix to stop testing unsupported PHP versions below 7.2.

Reviewed changes

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

File Description
README.md Updates stated minimum PHP version to match the new support floor.
composer.json Raises PHP requirement to >=7.2.0 and updates PHPUnit dev version constraints accordingly.
.github/workflows/ci.yml Removes PHP <7.2 from the CI test matrix to match the new minimum requirement.

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

@donatj donatj merged commit d890b4a into master May 15, 2026
19 checks passed
@donatj donatj deleted the donatj-patch-2 branch May 15, 2026 09:39
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.

2 participants