rfscholte commented on pull request #90:
URL: https://github.com/apache/maven-shade-plugin/pull/90#issuecomment-826086792
I'm afraid I can't convince most of you that this a bad proposal, and that
it is a lazy workaround for an issue somewhere else. Probably POM model 4.0.0
is missing an option to better control this so for only for that reason I'm
willing to allow this skip parameter.
However, as said before, having the skip parameter exposed as a property is
too dangerous.
I know a lot of people don't fully understand properties, so they are not
aware of the consequences.
For people that want to be able to control it from commandline could add
`<skip>${shade.skip}</skip>`, as can be done with any other parameter.
--
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:
[email protected]
