search

groovy / gmaven

Groovy integration for Maven
http://groovy.github.io/gmaven/
Apache License 2.0
51 stars 21 forks source link

Add skip parameter (#26) #27

Closed ppalaga closed 2 years ago

ppalaga commented 2 years ago

@ppalaga curious did you have a use-case for skip on exec then or is the change purely theoretical that goals should have a skip flag?

Yes, we use gmaven:execute for generating some product metadata, for generating parts of documentation and for various source checks. We have all affected files in git, so if a developer is building the main branch in a state that was previously verified by the CI, there is no point in performing those time consuming gmaven tasks.

please do remove the skips from the other goals and I think we can merge the PR with the exec skip feature.

Thanks, done in acd86c6

jdillon commented 2 years ago

@ppalaga thanks!