-
Notifications
You must be signed in to change notification settings - Fork 43
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
RAT-190: Fixes for minor bugs that blocked problem detection #250
Conversation
d116ccb
to
5b5f37f
Compare
5b5f37f
to
0c41c9c
Compare
return new ClaimReporterMultiplexer(configuration.isDryRun(), analysisMultiplexer, reporters); | ||
return new ClaimReporterMultiplexer(writer, configuration.isDryRun(), analysisMultiplexer, reporters); | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LHF: Pls reformat this class.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LHF? what does that mean?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sry, Low Hanging Fruit (LHF) - I use this abbreviation in commits or reviews in case of minor details or simple things.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fine to me. Thanks for your efforts.
@@ -160,7 +160,7 @@ public String desc() { | |||
* Name of File to exclude from report consideration. | |||
*/ | |||
static final Option EXCLUDE_CLI = Option.builder("e").longOpt("exclude").hasArgs().argName("Expression") | |||
.desc("Excludes files matching wildcard <expression>. May be followed by multiple arguments. " | |||
.desc("Excludes files matching wildcard <Expression>. May be followed by multiple arguments. " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pls update the site template to keep its text in sync with the stuff in the code. Should we add a javadoc in this class on where to keep it in sync with in case of changes?
This code is dependant upon #246 and should not be merged until that one is merged.
This is not a fix for Rat-190 as that issue was solved with the Tika introduction.
This change fixes some issues with the current source code.