search

vmware-archive / haret

A strongly consistent distributed coordination system, built using proven protocols & implemented in Rust.
461 stars 18 forks source link

Add `lint` target for Make #97

Closed jrgarcia closed 7 years ago

jrgarcia commented 7 years ago

Add lint target to Makefile. This currently expects to have clippy installed. If it is not, it errors out as you would expect, but also echos a message with instructions on how to install clippy.

Closes #23

jrgarcia commented 7 years ago

This was the best option I could find. Using clippy as an optional dependency and feature didn't return the same results on every run and got hung up on one occasion.

Also, I suppressed the full command to run clippy and added an @echo with just cargo clippy to appear as though it is just running cargo clippy (which it ultimately is unless not available).

andrewjstone commented 7 years ago

Thanks @jrgarcia. This works like a charm!

andrewjstone commented 7 years ago

Also, looks like I have quite a few things to cleanup :)