Skip to content

Update test environment and dependencies#40

Merged
clue merged 1 commit into
clue:1.xfrom
clue-labs:update-tests
Jun 29, 2026
Merged

Update test environment and dependencies#40
clue merged 1 commit into
clue:1.xfrom
clue-labs:update-tests

Conversation

@clue

@clue clue commented Jun 29, 2026

Copy link
Copy Markdown
Owner

This changeset updates the test environment and bumps the dependency minimums.

Builds on top of #39, #36, reactphp/stream#179, reactphp/event-loop#282, clue/framework-x#297, and clue/reactphp-eventsource#35

@clue clue added this to the v1.4.0 milestone Jun 29, 2026
@clue clue requested a review from Copilot June 29, 2026 10:43

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 the project’s test environment and raises the minimum supported versions for core ReactPHP dependencies, aligning CI and documentation with the updated setup.

Changes:

  • Bump dependency constraints to react/stream ^1.4 and react/event-loop ^1.6.
  • Update GitHub Actions CI to run on ubuntu-24.04 and use actions/checkout@v7.
  • Document how to generate a local text coverage report with Xdebug (XDEBUG_MODE=coverage …).

Reviewed changes

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

File Description
README.md Adds instructions for running PHPUnit with coverage locally and adjusts badge ordering.
composer.json Raises minimum versions for ReactPHP stream and event-loop dependencies.
.github/workflows/ci.yml Updates CI runner image and checkout action version while keeping the existing coverage enforcement.

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

@clue clue merged commit 60c4f98 into clue:1.x Jun 29, 2026
16 checks passed
@clue clue deleted the update-tests branch June 29, 2026 10:50
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