Claudenw commented on code in PR #246:
URL: https://github.com/apache/creadur-rat/pull/246#discussion_r1590328305
##########
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:
We could do so and provide a descriptoin. We don't need the location as
using `-s missing-headers` is all that is needed to specify the internal XSLT
files. We might note where they are in comments so that developers can find
them easily. Or point to them from documentation for XSLT development.
--
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
