McFoggy commented on pull request #565: URL: https://github.com/apache/maven/pull/565#issuecomment-940060099
Hi, as you ping me there for the replacement of #437 and #438 here is what I can tell you. When I created the PRs, I was looking for a standardized way to use & inject "Logger" (whatever the interface/impl) in different maven places. Standard plugins used to inherit from Mojo and thus access a Log interface from Mojo/AbstractMojo but for core extensions (what I was more interested of) it was not clear what to use and how. Looking at this PR it looks like the answer is now `inject and use LogFactory`. Then more specifically for #438 the introduction of the functional providers of message was more to allow new syntax of writing logs. This PR provides new Log methods but not functional ones allowing to use lambda or method references. For me #438 it is still valid or could be merged here with the other enhancements to Log interface. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
