Skip to content

Adds support for psr/log v3 (breaking change so new major version suggested)#359

Closed
lombo wants to merge 4 commits intophp-coveralls:masterfrom
lombo:v3.0.0
Closed

Adds support for psr/log v3 (breaking change so new major version suggested)#359
lombo wants to merge 4 commits intophp-coveralls:masterfrom
lombo:v3.0.0

Conversation

@lombo
Copy link
Copy Markdown

@lombo lombo commented Feb 11, 2023

Description

This PR adds support for psr/log v3. As it needs PHP 8.1+ this PR drops as well previous PHP versions.

It also takes advantage to migrate all PHPUnit tests to be compatible with PHPUnit 9.3+ and removes anmai/phpunit-legacy-adapter. Migrating tests needed phpspec/prophecy-phpunit as it will be deprecated from PHPUnit 10.0.

As it breaks backwards compatibility, a major version release is suggested.

TODO

  • Update CHANGELOG.

PRs/Issues to be closed after merge

We add support for psr/log v3.

It needs PHP 8.1+ so we dropped support for previous PHP versions. This means we aldo dropped support for old versions of other dependencies.
@lombo lombo changed the title Adds support for psr/log v3 (breaking changes so new major version suggested) Adds support for psr/log v3 (breaking change so new major version suggested) Feb 11, 2023
@MGatner
Copy link
Copy Markdown

MGatner commented Sep 27, 2023

Keen on this PR as this library is blocking our update to psr/log v3. Any status updates? Would it make sense in the meantime to send a smaller PR that simply updates composer.json to psr/log: ^1.0 || ^2.0 || ^3.0? I don't see any code issues that this would impact.

@MGatner
Copy link
Copy Markdown

MGatner commented Sep 27, 2023

Ah I see it would actually impact ConsoleLogger, but maybe there is a way around that.

@adrienfr
Copy link
Copy Markdown

Hi @lombo,

Thank you for your PR!

@keradus, do you think a new major version could be released anytime soon to include this change?
It seems a few other people are unable to use this package due to this issue (see #375).

I think composer.json should be rebased first, as the Symfony packages are outdated and don’t support Symfony 7.

Thanks a lot!

@keradus
Copy link
Copy Markdown
Member

keradus commented Nov 6, 2025

replaced by #402 , thank you @lombo

@keradus keradus closed this Nov 6, 2025
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.

Readme doc's are out of date Drop support for php version lower than 7.2

4 participants