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

[4.2] Annotate Psr\Log\LoggerInterface as provided by LogServiceProvider. #5965

Merged
merged 1 commit into from
Oct 16, 2014
Merged

[4.2] Annotate Psr\Log\LoggerInterface as provided by LogServiceProvider. #5965

merged 1 commit into from
Oct 16, 2014

Conversation

spencerdeinum
Copy link
Contributor

The Psr\Log\LoggerInterface was added to app->bind but not added to the
provides array in the LogServiceProvider.

See: itsgoingd/clockwork#67

The Psr\Log\LoggerInterface was added to app->bind but not added to the
provides array in the LogServiceProvider.

See: itsgoingd/clockwork#67
@spencerdeinum spencerdeinum changed the title Annotate Psr\Log\LoggerInterface as provided by LogServiceProvider. [4.2] Annotate Psr\Log\LoggerInterface as provided by LogServiceProvider. Oct 6, 2014
@GrahamCampbell
Copy link
Member

👍

@YOzaz
Copy link

YOzaz commented Oct 12, 2014

👍 yup.

taylorotwell added a commit that referenced this pull request Oct 16, 2014
…e_provider

[4.2] Annotate Psr\Log\LoggerInterface as provided by LogServiceProvider.
@taylorotwell taylorotwell merged commit 2bdd744 into laravel:4.2 Oct 16, 2014
@sebastiaanluca
Copy link
Contributor

Any ETA on a 4.x release with this fix included?

@GrahamCampbell
Copy link
Member

I don't know if any of the 4.2 bugs are blocking the tag.

@spencerdeinum
Copy link
Contributor Author

I'm still pinned at 4.2.9 until a new release is cut.

@GrahamCampbell
Copy link
Member

No, we're at v4.2.11: https://github.com/laravel/framework/releases. Related issue: #6200.

@GrahamCampbell
Copy link
Member

Oh, do you mean you can't upgrade until this is included in a tag?

@sebastiaanluca
Copy link
Contributor

Same here as @spencerdeinum, 4.2.11 doesn't have the fix included and that release has been pushed a while ago now.

@GrahamCampbell: yep :)

@driesvints
Copy link
Member

@quagh @spencerdeinum @GrahamCampbell FYI: Taylor just tagged 4.2.12 which will contain this bugfix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants