MartinKanters commented on pull request #85:
URL: https://github.com/apache/maven-shared-utils/pull/85#issuecomment-826547997


   > > Did you remove the default messages on these methods on purpose? If so, 
don't you think it adds value?
   > 
   > The remarks like "By default, bold cyan" refer to one particular 
implementation (the `AnsiMessageBuilder`) but they may not apply to other 
implementations (such as the `PlainMessageBuilder`). The interface should 
refrain from describing such implementation details, I would argue.
   
   Right, makes sense to me!


-- 
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:
us...@infra.apache.org


Reply via email to