ottlinger commented on code in PR #556:
URL: https://github.com/apache/creadur-rat/pull/556#discussion_r2453343605
##########
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:
@Claudenw Introducing System.exit(1) introduces problems as our testing
framework does not properly handle JVM exits via ReportTest. If this is an
issue we should open a separate ticket and maybe introduce
https://github.com/tginsberg/junit5-system-exit. Feel free to file a new ticket
with a reference to RAT-500 and your comment. Thanks
--
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]
