ottlinger commented on code in PR #246:
URL: https://github.com/apache/creadur-rat/pull/246#discussion_r1590322105
##########
apache-rat-core/src/main/java/org/apache/rat/Report.java:
##########
@@ -59,74 +65,166 @@
import org.apache.rat.walker.ArchiveWalker;
import org.apache.rat.walker.DirectoryWalker;
+import static java.lang.String.format;
+
/**
* The CLI based configuration object for report generation.
*/
public class Report {
+
+ private static final String[] NOTES = {
+ "Rat highlights possible issues.",
+ "Rat reports require interpretation.",
+ "Rat often requires some tuning before it runs well against a
project.",
+ "Rat relies on heuristics: it may miss issues"
+ };
+
+ private static final String[] STYLE_SHEETS = {"plain-rat",
"missing-headers", "unapproved-licenses"};
Review Comment:
Should we extract this into an enum -
apart from the text/CLI option value it could also contain the location of
the corresponding XSLT and a description.
WDYT?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@creadur.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
