Skip to content

Bumping PHP to supported version from 8.0 to 8.2 + dependencies#343

Merged
escopecz merged 7 commits intomautic:mainfrom
escopecz:fixing-cs-fix
Jan 27, 2026
Merged

Bumping PHP to supported version from 8.0 to 8.2 + dependencies#343
escopecz merged 7 commits intomautic:mainfrom
escopecz:fixing-cs-fix

Conversation

@escopecz
Copy link
Member

@escopecz escopecz commented Jan 26, 2026

I can see this warning in the CI that has now started to fail:

You are running PHP CS Fixer on PHP 8.3.29, but the minimum PHP version supported by your project in composer.json is PHP 8.0. Executing PHP CS Fixer on newer PHP versions may introduce syntax or features not yet available in PHP 8.0, which could cause issues under that version. It is recommended to run PHP CS Fixer on PHP 8.0, to fit your project specifics.

That highlights 2 issues. One is that this library is still supporting a version that is 2 years out of security updates. We should upgrade that to 8.2 which still gets security updates:

https://www.php.net/supported-versions.php

While messing with dev dependencies I increased versions for CS fixer (that allowed me to see and automatically fix the issues from the CI), PHPSTAN and added Rector with default config. Plus adding a step for CI to run it.

Copy link
Member

@patrykgruszka patrykgruszka left a comment

Choose a reason for hiding this comment

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

Isn't this a BC? I think we should update the requirements in README.md and bump the major version of this lib.

Rest of the changes are looking good.

@escopecz
Copy link
Member Author

Yes, we are planning to release V4 as requested in #316 (comment) as the v4-beta is released for a year now. It's a good opportunity to update. I updated Readme.

Copy link
Member

@RCheesley RCheesley left a comment

Choose a reason for hiding this comment

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

Looks good to go from my side, readme comment is clear. Thanks @escopecz!

@escopecz escopecz merged commit 0746a09 into mautic:main Jan 27, 2026
2 checks passed
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