Skip to content

Let go of PHP versions 7.4, 8.0, and 8.1#237

Merged
sanmai merged 3 commits intomasterfrom
sanmai/php8.2
Dec 4, 2025
Merged

Let go of PHP versions 7.4, 8.0, and 8.1#237
sanmai merged 3 commits intomasterfrom
sanmai/php8.2

Conversation

@sanmai
Copy link
Collaborator

@sanmai sanmai commented Dec 2, 2025

Removed PHP versions 7.4, 8.0, and 8.1 from the workflow matrix and updated the include section to reflect only PHP 8.2 with lowest dependencies.

  • I've run the tests with vendor/bin/phpunit
  • None of the tests were found failing
  • I've seen the coverage report at build/coverage/index.html
  • Not a single line left uncovered by tests
  • Any coding standards issues were fixed with vendor/bin/php-cs-fixer fix

Removed PHP versions 7.4, 8.0, and 8.1 from the workflow matrix and updated the include section to reflect only PHP 8.2 with lowest dependencies.
Copy link

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 PR drops support for older PHP versions (7.4, 8.0, and 8.1), establishing PHP 8.2 as the new minimum supported version. This aligns the project with actively supported PHP versions and allows leveraging modern PHP features.

Key Changes:

  • Updated minimum PHP version requirement from ^7.4 || ^8.0 to ^8.2 in composer.json
  • Removed PHP 7.4, 8.0, and 8.1 from the CI test matrix
  • Updated the lowest dependencies test to run on PHP 8.2 instead of 7.4

Reviewed changes

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

File Description
composer.json Updated PHP requirement to ^8.2, establishing the new minimum supported version
.github/workflows/tests.yaml Removed PHP 7.4, 8.0, and 8.1 from test matrix; updated lowest dependencies test to PHP 8.2

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

@sanmai sanmai merged commit 69dfccb into master Dec 4, 2025
5 checks passed
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