Skip to content

Conversation

@bensinclair
Copy link
Member

This PR adds PHP 8.4 to tests.

@bensinclair bensinclair marked this pull request as ready for review July 15, 2025 05:56
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

Adds PHP 8.4 to the continuous integration test matrix, ensuring the project is validated against the upcoming PHP release.

  • Expands the php-versions matrix to include '8.4'
Comments suppressed due to low confidence (2)

.github/workflows/tests.yml:16

  • Make sure the CI uses a version of the setup-php action that supports PHP 8.4 to avoid installation errors.
        php-versions: [ '8.0', '8.1', '8.2', '8.3', '8.4' ]

.github/workflows/tests.yml:16

  • [nitpick] Adding PHP 8.4 increases the CI matrix size and overall runtime; consider using strategy.max-parallel or caching steps to keep builds fast.
        php-versions: [ '8.0', '8.1', '8.2', '8.3', '8.4' ]

@joshmcrae joshmcrae merged commit 65f4e6a into master Jul 15, 2025
5 checks passed
@joshmcrae joshmcrae deleted the php-8-4-tests branch July 15, 2025 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants