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 psr/log:^3 #1187

Closed
wants to merge 1 commit into from
Closed

Support psr/log:^3 #1187

wants to merge 1 commit into from

Conversation

Ocramius
Copy link

See php-fig/log@2.0.0...3.0.0

psr/log:3.0.0 adds return types to all signatures: we don't implement psr/log, but only consume it, so we should be fine here.

See php-fig/log@2.0.0...3.0.0

`psr/log:3.0.0` adds return types to all signatures: we don't implement `psr/log`, but only consume it, so we should be fine here.
@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@cla-checker-service
Copy link

cla-checker-service bot commented Dec 21, 2021

💚 CLA has been signed

@Ocramius
Copy link
Author

It's signed, but the CLA process verification craps out because of blacklisted domains that do network tracking, and are (rightfully) blocked by my own network rules.

Feel free to look it up.

@ruudk
Copy link

ruudk commented Dec 22, 2021

Duplicate of #1184 but I don't mind which one is merged :D

@franmomu franmomu mentioned this pull request Dec 31, 2021
@technige
Copy link

Closing, as duplicate of #1184. Please check for existing PRs before submitting a new one.

@technige technige closed this Jan 20, 2022
@Ocramius Ocramius deleted the patch-1 branch January 20, 2022 13:31
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.

None yet

4 participants