Skip to content

Allow use of Monolog 2.x (as well as 1.x) - #64

Merged
miccheng merged 1 commit into
CoderKungfu:masterfrom
ejegg:monolog2
Apr 9, 2021
Merged

Allow use of Monolog 2.x (as well as 1.x)#64
miccheng merged 1 commit into
CoderKungfu:masterfrom
ejegg:monolog2

Conversation

@ejegg

@ejegg ejegg commented Apr 9, 2021

Copy link
Copy Markdown
Collaborator

No changes needed to the Logger class itself, as the StreamHandler
constructor is compatible in 2.x. Usages of non-PSR-standard e.g.
Logger->addInfo() methods are replaced with PSR-standard e.g.
Logger->info() calls, as both exist in 1.x and the former are
removed in Monolog 2.

See https://github.com/Seldaek/monolog/blob/main/UPGRADE.md

No changes needed to the Logger class itself, as the StreamHandler
constructor is compatible in 2.x. Usages of non-PSR-standard e.g.
Logger->addInfo() methods are replaced with PSR-standard e.g.
Logger->info() calls, as both exist in 1.x and the former are
removed in Monolog 2.

See
https://github.com/Seldaek/monolog/blob/main/UPGRADE.md
@miccheng

miccheng commented Apr 9, 2021

Copy link
Copy Markdown
Member

Looks good. Thanks. I'll have to fix the Travis-CI bits later.

@miccheng
miccheng merged commit 0f54d41 into CoderKungfu:master Apr 9, 2021
@ejegg

ejegg commented Apr 9, 2021

Copy link
Copy Markdown
Collaborator Author

Thanks so much for the speedy merge, @miccheng !

If it's not too much trouble, would you be able to tag a version 1.0.1 and update the version in packagist?

@miccheng

miccheng commented Apr 9, 2021

Copy link
Copy Markdown
Member

Done!

@ejegg

ejegg commented Apr 10, 2021

Copy link
Copy Markdown
Collaborator Author

Thanks again @miccheng !

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.

2 participants