search

pagevamp / laravel-cloudwatch-logs

Cloud Watch Driver for Laravel as needed at pagevamp
93 stars 48 forks source link

Monolog 2.0 compatibility #18

Closed particleflux closed 4 years ago

particleflux commented 4 years ago

With the release of v2.0.0 of maxbanton/cwh , that lib is now Monolog 2.0 compatible.

Would be awesome if that could be supported here as well :)

developernaren commented 4 years ago

https://github.com/pagevamp/laravel-cloudwatch-logs/pull/17 I created this PR for Larvael 7. it includes monolog 2 upgrade. Can you try this out?

particleflux commented 4 years ago

Seems to work, despite the fact that you seem to have missed a place to update the disabled flag. I added a comment to the PR.

Tested with laravel 6.2 and monolog 2

developernaren commented 4 years ago

Release 0.2.0 with these changes. @particleflux thanks for testing and PR review.

particleflux commented 4 years ago

Thanks for the fix :+1: