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

Removed console appender #88#89

Open
tdziurko wants to merge 1 commit into4finance:masterfrom
tdziurko:issues/88-remove-console-appender
Open

Removed console appender #88#89
tdziurko wants to merge 1 commit into4finance:masterfrom
tdziurko:issues/88-remove-console-appender

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