AdamBien/wad

Use `verify` instead of `clean install`

t1 opened this issue · 2 comments

t1 commented

As Robert Scholte keeps repeating, using clean install is a bad habit from the past ;-)

clean is not required any more by almost all plugins, as they support incremental builds. You rob them from their chance to optimize by blindly forcing them to redo everything.

install is also not necessary if you only need a verified artifact in your target folder. Just use verify and skip the installation to your local .m2 repository.

This would be a minimal change, but the speed improvement can be small but noticeable when doing it several times a minute.

AdamBien commented

+1 I will target that for the next release

sharpedavid commented

As Robert Scholte keeps repeating, using clean install is a bad habit from the past ;-)

clean is not required any more by almost all plugins, as they support incremental builds. You rob them from their chance to optimize by blindly forcing them to redo everything.

install is also not necessary if you only need a verified artifact in your target folder. Just use verify and skip the installation to your local .m2 repository.

This would be a minimal change, but the speed improvement can be small but noticeable when doing it several times a minute.

Thanks for the tip! I had no idea I should not be cleaning and installing so often. It makes sense when you put it like that.