[CI] Remove the phpunit 13.4 conflict, require Codeception 5.3.7 by mcop1 · Pull Request #4161 · pimcore/studio-ui-bundle · GitHub
Skip to content

[CI] Remove the phpunit 13.4 conflict, require Codeception 5.3.7 - #4161

Merged
mcop1 merged 1 commit into
2025.4from
fix/remove-phpunit-13-4-conflict
Oct 5, 2026
Merged

mcop1 merged 1 commit into
2025.4from
fix/remove-phpunit-13-4-conflict

Conversation

@mcop1

@mcop1 mcop1 commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Changes in this pull request

Related: Codeception/Codeception#6952
Follow-up of #4158.

Codeception 5.3.7 fixes the PHPUnit 13.4 incompatibility (Codeception/Codeception#6953). It passes the Emitter to Cli\Builder and Registry::init() on PHPUnit ≥ 13.4, and keeps the old calls below that. So the temporary pin can go.

  • Removed "phpunit/phpunit": ">=13.4" from conflict.
  • Raised require-dev codeception/codeception from ^5.3.5 to ^5.3.7. This repository's own CI then can't combine PHPUnit 13.4 with a Codeception that lacks the fix, including the lowest legs.

Dependents: the conflict is gone, so bundles and projects are unrestricted again. Their highest legs resolve Codeception 5.3.7, and their lowest legs stay on PHPUnit 12.5. No release ever shipped the conflict, so there is no released constraint to clean up.

Additional info

  • Verification:
    • composer validate passes.
    • Dry-run resolution, highest: codeception 5.3.7 + phpunit 13.4.1.
    • Dry-run resolution, --prefer-lowest: codeception 5.3.7 + phpunit 12.5.23.
    • The Codeception legs in CI on this PR are the real check. I did not run the suite locally.
  • After merging: forward-merge 2025.4 → 2026.3 → 2026.x. Both carry the same conflict.
  • Siblings: pimcore/pimcore 2026.3 and ee-pimcore 12.3 get the same change in separate PRs.
  • No regression test: this changes only which dependency versions may be installed.

🤖 Generated with Claude Code

Codeception 5.3.7 supports PHPUnit 13.4 (Codeception/Codeception#6953,
fixes #6952), so the temporary `conflict` "phpunit/phpunit": ">=13.4"
is no longer needed. Raise the require-dev floor to ^5.3.7 so this
repository's own CI never pairs PHPUnit 13.4 with an older Codeception.

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 5, 2026 07:26
@mcop1 mcop1 added this to the 2026.3.1 milestone Oct 5, 2026
@mcop1 mcop1 self-assigned this Oct 5, 2026
@sonarqubecloud

sonarqubecloud Bot commented Oct 5, 2026

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approved

The dependency constraints align with the verified Codeception 5.3.7 compatibility release.

Review effort: Balanced
Findings: None

What changed in this PR

Updates test dependencies after Codeception 5.3.7 restored PHPUnit 13.4 compatibility.

Changes:

  • Requires Codeception 5.3.7 or later.
  • Removes the temporary PHPUnit 13.4 conflict.
File Description
composer.json Updates Codeception and restores PHPUnit 13.4 availability.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mcop1 mcop1 added Task Internal Skip Milestone Check Exempts this PR from the mandatory milestone requirement labels Oct 5, 2026
@mcop1
mcop1 merged commit 7411945 into 2025.4 Oct 5, 2026
13 of 15 checks passed
@mcop1
mcop1 deleted the fix/remove-phpunit-13-4-conflict branch October 5, 2026 07:34
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Internal Skip Milestone Check Exempts this PR from the mandatory milestone requirement Task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants