Skip to content

Fix PHPUnit 9.x compatibility#2368

Merged
maks-rafalko merged 5 commits into
infection:masterfrom
staabm:fix-ci
Aug 13, 2025
Merged

Fix PHPUnit 9.x compatibility#2368
maks-rafalko merged 5 commits into
infection:masterfrom
staabm:fix-ci

Conversation

@staabm

@staabm staabm commented Aug 13, 2025

Copy link
Copy Markdown
Contributor

the tested started failling because with #2366 we no longer allow old sebastian/diff releases which in turn are required for old phpunit (unintentional side-effect)

Comment thread composer.json Outdated
"sanmai/later": "^0.1.7",
"sanmai/pipeline": "^7.0",
"sebastian/diff": "^6.0 || ^7.0",
"sebastian/diff": "^5.0 || ^6.0 || ^7.0",

@staabm staabm Aug 13, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

if we still want to support phpunit 9.x we would have to re-add also ^4.0... are we fine with dropping 9.x which is "End of Bugfix Support" since February 2, 2024?

see https://phpunit.de/supported-versions.html

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Just checked installation stats on packagist.org.

It seems 9.x is the most widely used version and we should still support it

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm personally ok with either ways. If we decide to not support 9.x, developers can use older Infection versions.

At least this logic is applied when we got read of non-supported PHP versions each year

@staabm staabm Aug 13, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Will add back 9.x support later today. There is no benefit in dropping it atm

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

cool, thanks. This should also fix #2367

@staabm staabm changed the title Fix "Without_Git" e2e test Fix PHPUnit 9.x compatibility Aug 13, 2025
@staabm

staabm commented Aug 13, 2025

Copy link
Copy Markdown
Contributor Author

should be good to go.

failling tests seem to be the ones which fail from time to time

@maks-rafalko maks-rafalko enabled auto-merge (squash) August 13, 2025 21:58
@maks-rafalko maks-rafalko merged commit 825526c into infection:master Aug 13, 2025
97 of 99 checks passed
@staabm staabm deleted the fix-ci branch August 14, 2025 05:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants