search

AdamBien / wad

Watch and Deploy - deploying ThinWARs without IDE plugins
http://wad.sh
MIT License
111 stars 24 forks source link

Use `verify` instead of `clean install` #7

Open t1 opened 4 years ago

t1 commented 4 years ago

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 4 years ago

+1 I will target that for the next release

sharpedavid commented 4 years ago

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.