MartinKanters commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1180126255
##########
maven-embedder/src/main/java/org/apache/maven/cli/CLIManager.java:
##########
@@ -173,7 +177,17 @@ public CLIManager() {
.build());
options.addOption(Option.builder(Character.toString(BATCH_MODE))
.longOpt("batch-mode")
- .desc("Run in non-interactive (batch) mode (disables output
color)")
+ .desc(
+ "Run in non-interactive (batch) mode (disables output
color) - alias for --non-interactive (kept for backwards compatability)")
+ .build());
+ options.addOption(Option.builder(NON_INTERACTIVE)
+ .longOpt("non-interactive")
+ .desc("Run in non-interactive (batch) mode (disables output
color) - alias for --batch-mode")
+ .build());
+ options.addOption(Option.builder(FORCE_INTERACTIVE)
+ .longOpt("force-interactive")
+ .desc(
+ "Run in interactive mode - even when the environment
variable CI is set to true and --non-interactive or --batch-mode are set")
Review Comment:
I'm not sure how to pass this information to Jansi. `isatty(2)` will atm
just return 1/true. Also if `--batch-mode` is set.
Do you think a change like this should be part of this PR?
--
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: issues-unsubscr...@maven.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
