Skip to content

Support PHPUnit 13#342

Draft
jnoordsij wants to merge 2 commits intonunomaduro:v8.xfrom
jnoordsij:phpunit-v13
Draft

Support PHPUnit 13#342
jnoordsij wants to merge 2 commits intonunomaduro:v8.xfrom
jnoordsij:phpunit-v13

Conversation

@jnoordsij
Copy link
Contributor

@jnoordsij jnoordsij commented Feb 6, 2026

@aldemeery
Copy link

@jnoordsij Any updates on this?

@jnoordsij
Copy link
Contributor Author

Currently Pest seems to block installing this here; see https://github.com/pestphp/pest/blob/4.x/composer.json#L33. There might be even more upstream limitations.

@dbebawy
Copy link

dbebawy commented Feb 10, 2026

I've submitted a PR on the Pest side to unblock this:

Breaking the circular dependency

The current blocker chain is:

  1. Collision conflicts with PHPUnit ≥13.0.0
  2. Collision's CI uses Pest, which also doesn't support PHPUnit 13
  3. Pest requires collision, which conflicts with PHPUnit 13

The Pest PR breaks this from the Pest side. Once merged, collision can:

  1. Update the conflicts section to allow PHPUnit 13
  2. Run CI with PHPUnit 13 using Pest (once Pest releases with the change)

Interim suggestion for this PR

Since Pest is only a dev dependency of collision, this PR could also decouple the PHPUnit 13 CI from Pest:

  • Add a CI matrix job that runs collision's PHPUnit integration tests directly with PHPUnit 13 (without Pest)
  • Keep the existing Pest-based CI for PHPUnit 12
  • This allows merging collision's PHPUnit 13 support without waiting for the full Pest release

The runtime code changes in this PR can be validated independently of Pest.

@aldemeery
Copy link

@dbebawy Super! Thanks a lot!

@aldemeery
Copy link

Any updates on this?

CC @nunomaduro

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.

3 participants