Skip to content

Conversation

@rosier
Copy link
Contributor

@rosier rosier commented Jan 9, 2015

Q A
Bug fix? no
New feature? no
BC breaks? yes
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useless given that the base logger (Monolog) already implements PSR-3

@rosier rosier changed the title [3.0][Monolog] Use Psr\Log\LoggerInterface and remove deprecated methods [3.0][Monolog] Remove deprecated interface and deprecated methods Jan 10, 2015
@rosier
Copy link
Contributor Author

rosier commented Jan 10, 2015

Current PR has been updated

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be removed.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nvm, indeed

@fabpot
Copy link
Member

fabpot commented Jan 16, 2015

Thank you @rosier.

fabpot added a commit that referenced this pull request Jan 16, 2015
…ted methods (rosier)

This PR was squashed before being merged into the 3.0-dev branch (closes #13348).

Discussion
----------

[3.0][Monolog] Remove deprecated interface and deprecated methods

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | yes
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT

Commits
-------

0fa072c [3.0][Monolog] Remove deprecated interface and deprecated methods
@fabpot fabpot closed this Jan 16, 2015
@rosier rosier deleted the monolog-cleanup branch January 16, 2015 15:21
@fabpot fabpot mentioned this pull request Nov 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants