Added command line option for log-level #1751
Conversation
|
Do we want the progress bars on level that shows less than info? (WARN, ERROR) Users that select these levels probably actually only want warnings and errors Do we want to keep the progress bars on DEBUG and TRACE? They are barely visible anyway and due to #1719 the additional messages are cut off sometimes |
tsaglam
added
enhancement
# Conflicts: # cli/src/main/java/de/jplag/cli/CLI.java
|
There was a problem hiding this comment.
Readding my comment from above as requested changes, to not forget it before merging:
Do we want the progress bars on level that shows less than info? (WARN, ERROR) Users that select these levels probably actually only want warnings and errors
Do we want to keep the progress bars on DEBUG and TRACE? They are barely visible anyway and due to #1719 the additional messages are cut off sometimes
In the meeting we decided we dont want the bar on any level other than info
…g levels other than info
|
@TwoOfTwelve one conflict has to be solved before I can merge. |
# Conflicts: # cli/src/main/java/de/jplag/cli/options/CliOptions.java
…re/cliLogLevel # Conflicts: # cli/src/main/java/de/jplag/cli/logger/CliProgressBarProvider.java # cli/src/main/java/de/jplag/cli/logger/CollectedLogger.java # cli/src/main/java/de/jplag/cli/logger/JPlagLoggerBase.java
|
|
Added a cli option to set the log-level