Add support for skipping if skipTests is defined
leonard84 opened this issue · 2 comments
leonard84 commented
Since this plugin is primarily used for testing, it would be a sensible default to skip the execution if -DskipTests
is defined and not false
. Since the handling of skipTests
is done on its presence, it does not work to manually forward it to the configuration <skip>${skipTests}</skip>
so explicit support is necessary.
If someone wants to explicitly never skip the execution then they could set <skip>false</skip>
in the configuration, which should overrule the skipTests
parameter.
andidev commented
That is true, do you feel like making a PR? This should not be to hard to fix.
andidev commented
Released in version 3.2.0