kemitix/kemitix-checkstyle-ruleset
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

builder:DefaultRuleReadmeLoader: show 'reason' instead for disabled rules

Browse source
This commit is contained in:
Paul Campbell 2017-01-05 23:41:45 +00:00
parent b70b7c4a5a
commit 1fa27f2071
1 changed files with 15 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
builder/src/main/java/net/kemitix/checkstyle/ruleset/builder/DefaultRuleReadmeLoader.java
View file

@ -23,14 +23,21 @@ class DefaultRuleReadmeLoader implements RuleReadmeLoader {
@Override
public Stream<String> load(final Rule rule) {
try {
final Path resolve = templateProperties.getReadmeFragments()
.resolve(rule.getName() + ".md");
log.info("Loading fragment: {}", resolve);
return Stream.concat(Stream.of(String.format("%n#### [%s](%s)", rule.getName(), rule.getUri())),
Files.lines(resolve));
} catch (IOException e) {
return Stream.empty();
if (rule.isEnabled()) {
try {
final Path resolve = templateProperties.getReadmeFragments()
.resolve(rule.getName() + ".md");
log.info("Loading fragment: {}", resolve);
return Stream.concat(Stream.of(formatRuleHeader(rule)), Files.lines(resolve));
} catch (IOException e) {
return Stream.empty();
}
} else {
return Stream.of(formatRuleHeader(rule), "", rule.getReason());
}
}
private String formatRuleHeader(final Rule rule) {
return String.format("%n#### [%s](%s)", rule.getName(), rule.getUri());
}
}