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

addin slf4j on blank project #748

Merged
merged 1 commit into from
Aug 20, 2014
Merged

addin slf4j on blank project #748

merged 1 commit into from
Aug 20, 2014

Conversation

Turini
Copy link
Member

@Turini Turini commented Aug 19, 2014

No description provided.

@Turini Turini self-assigned this Aug 19, 2014
@garcia-jj
Copy link
Member

Did you mean log4j? :D

Sounds OK for me. Its nice to use log4j as slf4j impl because is described
in our docs.

And most devs prefer log4j instead of jul.
On Aug 19, 2014 2:08 PM, "Rodrigo Turini" notifications@github.com wrote:


You can merge this Pull Request by running

git pull https://github.com/caelum/vraptor4 addingLo4jOnBlank

Or view, comment on, or merge it at:

#748
Commit Summary

  • addin slf4j on blank project

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#748.

@Turini
Copy link
Member Author

Turini commented Aug 20, 2014

Yes, using log4j as implementation! thks

Turini added a commit that referenced this pull request Aug 20, 2014
@Turini Turini merged commit 804bee8 into master Aug 20, 2014
@Turini Turini deleted the addingLo4jOnBlank branch August 20, 2014 10:14
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.

2 participants