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.

Sign up for GitHub

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

Jump to bottom

Fix monolog channels #89

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix monolog channels #89

wants to merge 4 commits into from

Commits on Sep 10, 2015

  1. Load monolog services by default 

    And remove these services if Monolog is not available. This way,
loggers can use the `guzzle` channel assigned by the service definition
and not the `app` channel.
    @iambrosi
    iambrosi committed Sep 10, 2015
    Configuration menu
    Copy the full SHA
    ad67c78 View commit details
    Browse the repository at this point in the history

  2. Using logger service instead of monolog.logger 

    Monolog only assigns log channels to those services that depend on the
`logger` service, not the `monolog.logger`.
    @iambrosi
    iambrosi committed Sep 10, 2015
    Configuration menu
    Copy the full SHA
    7e6e493 View commit details
    Browse the repository at this point in the history

  3. Fixed tests

    @iambrosi
    iambrosi committed Sep 10, 2015
    Configuration menu
    Copy the full SHA
    2b03230 View commit details
    Browse the repository at this point in the history

  4. Fixed compatibility with PHP 5.3

    @iambrosi
    iambrosi committed Sep 10, 2015
    Configuration menu
    Copy the full SHA
    a4c05d4 View commit details
    Browse the repository at this point in the history