ottlinger commented on code in PR #224:
URL: https://github.com/apache/creadur-rat/pull/224#discussion_r1536663580
##########
apache-rat-core/src/main/java/org/apache/rat/Report.java:
##########
@@ -303,15 +314,13 @@ static Options buildOptions() {
opts.addOption(null, SCAN_HIDDEN_DIRECTORIES, false, "Scan hidden
directories");
OptionGroup addLicenseGroup = new OptionGroup();
- String addLicenseDesc = "Add the default license header to any file
with an unknown license that is not in the exclusion list. "
- + "By default new files will be created with the license
header, "
- + "to force the modification of existing files use the --force
option.";
-
// RAT-85/RAT-203: Deprecated! added only for convenience and for
backwards
// compatibility
- Option addLicence = new Option(ADD_OLD, "addLicence", false,
addLicenseDesc);
+ Option addLicence = new Option(ADD_OLD, false, "(deprecated) Add the
default license header to any file with an unknown license. Use '-A' or
---addLicense instead.");
Review Comment:
Should we rename the internal variable to ADD_DEPRECATED (instead of OLD)?
--
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
