airlift / airbase

Base POM for Airlift
Apache License 2.0
48 stars 87 forks source link

Enable check plugins unless air.check.skip-all is set #301

Closed nineinchnick closed 2 years ago

nineinchnick commented 2 years ago

Profile activation is performed by Maven before resolving properties, so inherited properties cannot be used for it. To optimize for the most common scenario, only completely disable check plugins when all are skipped. When single plugins are skipped, it will be done in their configuration.

ksobolew commented 2 years ago

Additionally, these are not Maven properties but system properties. So changing the properties section to have <air.check.skip-all>true</air.check.skip-all> won't change the set of active profiles. It will only work when Maven is invoked with -Dair.check.skip-all=true.