kriegaex commented on pull request #104:
URL: 
https://github.com/apache/maven-shade-plugin/pull/104#issuecomment-883801321


   @rmannibucau, thanks for the new PR. Thank you so much for letting code 
speak and presenting an alternative solution instead of simply blocking this PR 
and calling it inadequate. I quickly looked at (not formally reviewed) the code 
and am happily conceding that your solution indeed eliminates the irritating 
warning in a much more fundamental way than my quick fix, namely by adding the 
new feature of handling input directories and input JARs uniformly with regard 
to service provider minification. Therefore, #108 is clearly superior to this 
PR, which is why I am closing it in favor of yours.


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