Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add helpful texts to logger config options #1990

Merged
merged 2 commits into from Mar 13, 2017

Conversation

Projects
None yet
3 participants
Member

richvdh commented Mar 13, 2017

Some comments in the logger config settings and cmdline params, to help guide users.

Also remove the INFO level on the file appender in the default logger config, which mostly served to confuse people when they expected to get debug logs.

Owner

erikjohnston commented Mar 13, 2017

lgtm

@richvdh richvdh merged commit 31f3ca1 into develop Mar 13, 2017

5 checks passed

Sytest Dendron (Merged PR) Build finished.
Details
Sytest Postgres (Merged PR) Build finished.
Details
Sytest SQLite (Merged PR) Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@lampholder lampholder removed the in progress label Mar 13, 2017

@richvdh richvdh deleted the rav/log_config_comments branch Mar 15, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment