Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Monolog 2 #53

Merged
merged 4 commits into from
Feb 14, 2020
Merged

Support Monolog 2 #53

merged 4 commits into from
Feb 14, 2020

Conversation

rvanlaak
Copy link
Collaborator

@rvanlaak rvanlaak commented Feb 11, 2020

fixes #52 but requires #55 to be merged first

All logger usages comply to psr/log, so we can safely support Monolog 2.

Did also check the Monolog UPGRADE notes to be sure.

RFC

  • I'm in favor of bumping our PHP requirement from ^7.1.3 to ^7.2.5

@rvanlaak rvanlaak self-assigned this Feb 11, 2020
@rvanlaak rvanlaak force-pushed the monolog-2 branch 2 times, most recently from ec7de7d to 1958a10 Compare February 11, 2020 15:53
Repository owner deleted a comment from coveralls Feb 11, 2020
@rvanlaak rvanlaak mentioned this pull request Feb 13, 2020
# Conflicts:
#	src/Micrometa/Tests/Infrastructure/ExceptionLoggerTest.php
@rvanlaak rvanlaak merged commit ef0089c into jkphl:master Feb 14, 2020
@rvanlaak rvanlaak deleted the monolog-2 branch February 14, 2020 10:07
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.

Bump monolog up to ^2
1 participant