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