Skip to content
This repository has been archived by the owner on May 11, 2021. It is now read-only.

Removed console appender #88 #89

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tdziurko
Copy link
Contributor

@tdziurko tdziurko commented Jun 5, 2015

No description provided.

@marcingrzejszczak
Copy link
Contributor

+1

@coveralls
Copy link

Coverage Status

Coverage remained the same at 78.91% when pulling f7c9f2c on tdziurko:issues/88-remove-console-appender into 837d1cd on 4finance:master.

@tdziurko
Copy link
Contributor Author

tdziurko commented Jun 5, 2015

Should I merge it to master? :)

@szpak
Copy link
Contributor

szpak commented Jun 5, 2015

A drawback is that it makes harder to work from IDE. No logs on console. It would be good (one sunny day - as a separate issue) to have it configurable and use a console in a development mode (-Dfoo=.. set from an Idea run menu).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants