Skip to content

Run tests on PHP 8.5 and update test environment#54

Merged
clue merged 1 commit into
clue:1.xfrom
clue-labs:php8.5
Jun 21, 2026
Merged

Run tests on PHP 8.5 and update test environment#54
clue merged 1 commit into
clue:1.xfrom
clue-labs:php8.5

Conversation

@clue

@clue clue commented Jun 21, 2026

Copy link
Copy Markdown
Owner

This changeset updates the test environment to run the test suite on PHP 8.5.

Builds on top of #53, #50, reactphp/event-loop#282, reactphp/promise#264, and clue/framework-x#297

@clue clue added this to the v1.8.0 milestone Jun 21, 2026
@clue clue requested a review from Copilot June 21, 2026 19:10

Copilot AI left a comment

Copy link
Copy Markdown

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 updates this library’s CI/test environment to ensure the test suite runs successfully on PHP 8.5, aligning dependency constraints with upstream releases that include PHP 8.5 compatibility fixes.

Changes:

  • Add PHP 8.5 to the GitHub Actions test matrix.
  • Tighten dependency constraints to minimum versions that include PHP 8.5+ compatibility fixes (notably react/promise, react/async, and react/event-loop).
  • Update the GitHub Actions checkout action to actions/checkout@v7.

Reviewed changes

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

File Description
composer.json Bumps ReactPHP dependency constraints to versions that include PHP 8.5+ support while retaining compatibility ranges for older versions.
.github/workflows/ci.yml Adds PHP 8.5 to CI and updates the checkout action version used by the workflow.

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

@clue clue merged commit c10407c into clue:1.x Jun 21, 2026
15 checks passed
@clue clue deleted the php8.5 branch June 21, 2026 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants