Skip to content

Monolog #174

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
Merged
merged 5 commits into from
Apr 4, 2011
Merged

Monolog #174

merged 5 commits into from
Apr 4, 2011

Conversation

@stof
Copy link
Member

@stof stof commented Mar 20, 2011

This adds the doc for MonologBundle.

I would appreciate some proof-reading of it.
ajessu
ajessu reviewed Mar 20, 2011
View reviewed changes
cookbook/logging/monolog.rst Outdated
Copy link
Contributor

@ajessu ajessu Mar 20, 2011

Choose a reason for hiding this comment

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

This should be:
the order matters and a merge does not...
or
the order matters and merging does not...
@weaverryan
Copy link
Member

weaverryan commented Mar 20, 2011

Very very complete as always stof - love seeing that you included the dic tags section and bundle configuration. Once you've made a few of the minor changes and are ready, I'll review it fully and merge it in.

Looks really good though - thanks!
@stof
Copy link
Member Author

stof commented Mar 20, 2011

I will had a commit with the changes. I hope Seldaek will proof-read it too as he is the best one to check if the doc about Monolog itself is right. I have mainly worked on the bundle.

Also wait for MonologBundle to be merged before merging the doc :)
@fabpot fabpot merged commit 12dbb42 into symfony:master Apr 4, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants

@stof @weaverryan @ajessu @fabpot