Claudenw commented on code in PR #556:
URL: https://github.com/apache/creadur-rat/pull/556#discussion_r2452663138
##########
apache-rat-core/src/main/java/org/apache/rat/Report.java:
##########
@@ -40,6 +40,12 @@ public final class Report {
* @throws Exception on error.
*/
public static void main(final String[] args) throws Exception {
+ if (args == null || args.length == 0) {
+ DefaultLog.getInstance().info("Please use the \"--help\" option to
see a " +
+ "list of valid commands and options, as you did not
provide any arguments.");
+ System.exit(0);
+ }
+
DefaultLog.getInstance().info(new VersionInfo().toString());
Review Comment:
Move this line above your new check as this one puts out the version info.
##########
apache-rat-core/src/main/java/org/apache/rat/Report.java:
##########
Review Comment:
I know this is not part of the same issue but, the information has already
been logged. So lets replace this throw with a System.exit(1)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
