Skip to content

Conversation

alcaeus
Copy link
Member

@alcaeus alcaeus commented Sep 25, 2025

PHPLIB-1709

For now, this includes a commit to test with the development version on the driver, as stable does not support PHP 8.5 yet.

@alcaeus alcaeus force-pushed the phplib-1709-test-php-8.5 branch 2 times, most recently from 3776dce to f9fa589 Compare September 25, 2025 09:25
@alcaeus alcaeus force-pushed the phplib-1709-test-php-8.5 branch from f9fa589 to 4fa2df4 Compare October 2, 2025 12:00
@alcaeus alcaeus marked this pull request as ready for review October 2, 2025 12:00
@alcaeus alcaeus requested a review from a team as a code owner October 2, 2025 12:00
@alcaeus alcaeus requested review from Copilot and paulinevos October 2, 2025 12:00
Copy link
Contributor

@Copilot 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 adds support for testing with PHP 8.5 by updating the CI configuration to include PHP 8.5 in the test matrix and enabling platform requirement ignoring for dependencies that may not yet support this version.

  • Added PHP 8.5 to the test matrix in the GitHub Actions workflow
  • Introduced a new input parameter to conditionally ignore platform requirements for PHP 8.5
  • Updated the Composer installation step to skip platform requirements when testing with PHP 8.5

Reviewed Changes

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

File Description
.github/workflows/tests.yml Added PHP 8.5 to test matrix and enabled platform requirement ignoring
.github/actions/setup/action.yml Added ignore-platform-req input and updated composer options

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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.

1 participant