gnodet commented on PR #13136:
URL: https://github.com/apache/maven/pull/13136#issuecomment-5680380324

   Thanks for the fix. Worth noting that #12697 (and its dependency chain 
starting at #12695) is introducing new event loggers 
(`PlainExecutionEventLogger`, `RichBuildEventListener`, etc.) that replace 
`ExecutionEventLogger`. The reactor summary behaviour you're fixing here would 
need to be ported to those new loggers as well — or alternatively, this fix 
could be implemented there directly if the new chain lands first. Just flagging 
so the work isn't duplicated or lost.


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

Reply via email to