[MonologBridge] added integration with the console component #8167

Closed
wants to merge 3 commits into
from

Projects

None yet

3 participants

@Tobion
Member
Tobion commented May 30, 2013

This adds integration with the console component by adding ConsoleHandler and ConsoleFormatter which can be used to show log messages in the console output depending on the verbosity settings.
This PR splits the reusable classes from symfony/monolog-bundle#42 into the bridge.
It has been reviewed by @Seldaek

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #8033
License MIT
Doc PR will be done when this is used by the MonologBundle
Tobion added some commits May 30, 2013
@Tobion Tobion [MonologBridge] added integration with the console component 11396f7
@Tobion Tobion [MonologBridge] adjust composer.json for console integration
I also moved the http-kernel from require to suggest because it's optional. E.g. if you dont need the ChromePhpHandler. This is also done in the TwigBridge where all integrated components are optional dependencies
cde88cf
@Tobion Tobion [MonologBridge] updated changelog e8308d2
@Tobion Tobion referenced this pull request in symfony/monolog-bundle May 30, 2013
Merged

added possibility show log messages in console commands #42

5 of 5 tasks complete
@Seldaek
Member
Seldaek commented May 31, 2013

👍

@Tobion
Member
Tobion commented Jun 2, 2013

@fabpot I'd appreciate if this gets merged soon so I can finish this. Then I can also write documentation.

@fabpot
Member
fabpot commented Jun 2, 2013

I will merge PRs for 2.4 just after the release of 2.3 (tomorrow).

@fabpot fabpot added a commit that referenced this pull request Jun 4, 2013
@fabpot fabpot merged branch Tobion/monolog-console-integration (PR #8167)
This PR was squashed before being merged into the master branch (closes #8167).

Discussion
----------

[MonologBridge] added integration with the console component

This adds integration with the console component by adding ConsoleHandler and ConsoleFormatter which can be used to show log messages in the console output depending on the verbosity settings.
This PR splits the reusable classes from symfony/monolog-bundle#42 into the bridge.
It has been reviewed by @Seldaek

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #8033
| License       | MIT
| Doc PR        | will be done when this is used by the MonologBundle

Commits
-------

c0b5996 [MonologBridge] added integration with the console component
46b209b
@fabpot fabpot added a commit that closed this pull request Jun 4, 2013
@fabpot fabpot merged branch Tobion/monolog-console-integration (PR #8167)
This PR was squashed before being merged into the master branch (closes #8167).

Discussion
----------

[MonologBridge] added integration with the console component

This adds integration with the console component by adding ConsoleHandler and ConsoleFormatter which can be used to show log messages in the console output depending on the verbosity settings.
This PR splits the reusable classes from symfony/monolog-bundle#42 into the bridge.
It has been reviewed by @Seldaek

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #8033
| License       | MIT
| Doc PR        | will be done when this is used by the MonologBundle

Commits
-------

c0b5996 [MonologBridge] added integration with the console component
46b209b
@fabpot fabpot closed this in 46b209b Jun 4, 2013
@Tobion Tobion deleted the Tobion:monolog-console-integration branch Aug 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment