MartinKanters commented on code in PR #869:
URL: https://github.com/apache/maven/pull/869#discussion_r1026584591


##########
maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java:
##########
@@ -1438,6 +1446,33 @@ private MavenExecutionRequest populateRequest( 
CliRequest cliRequest, MavenExecu
         return request;
     }
 
+    private void disableInteractiveModeIfNeeded( final CliRequest cliRequest, 
final MavenExecutionRequest request )
+    {
+        CommandLine commandLine = cliRequest.getCommandLine();
+        if ( commandLine.hasOption( FORCE_INTERACTIVE ) )
+        {
+            return;
+        }
+
+        boolean runningOnCI = isRunningOnCI( cliRequest.getSystemProperties() 
);
+        if ( runningOnCI )
+        {
+            slf4jLogger.info( "Detected a CI build, because the environment 
variable CI equals \"true\". "
+                + "Disable this detection by removing that variable or adding 
--force-interactive." );

Review Comment:
   I agree! That is much clearer for the users. I will change it.



-- 
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

Reply via email to