Marcono1234 commented on PR #219:
URL: 
https://github.com/apache/maven-shade-plugin/pull/219#issuecomment-2144018082

   > The inline init case is not supposed to work
   
   But does it work currently, and are plugins relying on this? Because if 
Maven Plugin Plugin is changed to pick a different default value in the future 
in those cases, or if it starts documenting a default value which is not 
actually the default value, then I doubt that those changes will be made.
   
   Either way, I think this is getting a bit off-topic. Would you mind merging 
the changes here?
   Even if Maven Plugin Plugin is changed, that will happen at some point in 
the future, and until then the Shade plugin documentation would still be 
incomplete.
   
   If you want the Maven Plugin Plugin to change, could you or someone else 
please drive those changes, because I am neither familiar enough with Maven nor 
Maven plugin development or the internals, and I personally also doubt that 
such a change would be worth it given the compatibility risks.


-- 
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: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to