rmannibucau commented on a change in pull request #438:
URL: https://github.com/apache/maven/pull/438#discussion_r564558395
##########
File path:
maven-plugin-api/src/main/java/org/apache/maven/plugin/logging/Log.java
##########
@@ -63,6 +65,20 @@
*/
void debug( Throwable error );
+ /**
+ * Send a message to the user in the <b>debug</b> error level by computing
the message
+ * only when needed. The supplier will be called only if @see
#isDebugEnabled() is <b>true</b>.
+ *
+ * @param messageSupplier a non null Supplier of the message to use
+ */
+ default void debug( Supplier<String> messageSupplier )
Review comment:
We had that discussion on Artifact#setPath/getpath thread and it ended
up as a compromise to do both.
I'm not sure it is worth but I'm fine with this but implementing it in
implementation is the prerequisite on my side - not having it in the interface
is fine for me since it is not a breaking change until you upgrade your mojo
API and have a custom impl - which is fine.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
