cstamas commented on code in PR #1231:
URL: https://github.com/apache/maven/pull/1231#discussion_r1320853920


##########
maven-embedder/src/main/java/org/apache/maven/cli/transfer/AbstractMavenTransferListener.java:
##########
@@ -184,14 +189,19 @@ protected AbstractMavenTransferListener(PrintStream out) {
 
     @Override
     public void transferInitiated(TransferEvent event) {
+        String darkOn = MessageUtils.isColorEnabled() ? ANSI_DARK_SET : "";

Review Comment:
   If we want to cut logging (and I do agree with @elharo we log way too much), 
I'd just remove this method (make it no op). 
   
   Or just leave it for PUT (leave it as before), and do not log nothing for 
the rest.
   
   Reason is simple: if you have 5 reposes, you will have 5 "Downloading...." 
but only one "Downloaded", and the latter matter.



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