mirror of https://github.com/status-im/consul.git
Makefile: add vet target
Add a vet target in order to catch suspicious constructs
reported by go vet.
Vet has successfully detected problems in the past,
for example, see
c9333b1b9b
Some vet flags are noisy. In particular, the following flags
reports a large amount of generally unharmful constructs:
```
-assign: check for useless assignments
-composites: check that composite literals used field-keyed
elements
-shadow: check for shadowed variables
-shadowstrict: whether to be strict about shadowing
-unreachable: check for unreachable code
```
In order to skip running the flags mentioned above, vet is
invoked on a directory basis with `go tool vet .` since package-
level type-checking with `go vet` doesn't accept flags.
Hence, each file is vetted in isolation, which is weaker than
package-level type-checking. But nevertheless, it might catch
suspicious constructs that pose a real issue.
The vet target runs the following flags on the entire repo:
```
-asmdecl: check assembly against Go declarations
-atomic: check for common mistaken usages of the
sync/atomic package
-bool: check for mistakes involving boolean operators
-buildtags: check that +build tags are valid
-copylocks: check that locks are not passed by value
-methods: check that canonically named methods are canonically
defined
-nilfunc: check for comparisons between functions and nil
-printf: check printf-like invocations
-rangeloops: check that range loop variables are used correctly
-shift: check for useless shifts
-structtags: check that struct field tags have canonical format
and apply to exported fields as needed
-unsafeptr: check for misuse of unsafe.Pointer
```
Now and then, it might make sense to check the output of the
disabled flags manually.
For example, `VETARGS=-unreachable make vet` can detect several
lines of dead code that can be deleted, etc.
This commit is contained in:
parent
58806bc7bb
commit
fe54356d61
16
Makefile
16
Makefile
|
@ -1,5 +1,7 @@
|
||||||
DEPS = $(shell go list -f '{{range .TestImports}}{{.}} {{end}}' ./...)
|
DEPS = $(shell go list -f '{{range .TestImports}}{{.}} {{end}}' ./...)
|
||||||
PACKAGES = $(shell go list ./...)
|
PACKAGES = $(shell go list ./...)
|
||||||
|
VETARGS?=-asmdecl -atomic -bool -buildtags -copylocks -methods \
|
||||||
|
-nilfunc -printf -rangeloops -shift -structtags -unsafeptr
|
||||||
|
|
||||||
all: deps format
|
all: deps format
|
||||||
@mkdir -p bin/
|
@mkdir -p bin/
|
||||||
|
@ -17,6 +19,7 @@ deps:
|
||||||
test: deps
|
test: deps
|
||||||
./scripts/verify_no_uuid.sh
|
./scripts/verify_no_uuid.sh
|
||||||
go list ./... | xargs -n1 go test
|
go list ./... | xargs -n1 go test
|
||||||
|
@$(MAKE) vet
|
||||||
|
|
||||||
integ:
|
integ:
|
||||||
go list ./... | INTEG_TESTS=yes xargs -n1 go test
|
go list ./... | INTEG_TESTS=yes xargs -n1 go test
|
||||||
|
@ -29,10 +32,21 @@ format: deps
|
||||||
@echo "--> Running go fmt"
|
@echo "--> Running go fmt"
|
||||||
@go fmt $(PACKAGES)
|
@go fmt $(PACKAGES)
|
||||||
|
|
||||||
|
vet:
|
||||||
|
@go tool vet 2>/dev/null ; if [ $$? -eq 3 ]; then \
|
||||||
|
go get golang.org/x/tools/cmd/vet; \
|
||||||
|
fi
|
||||||
|
@echo "--> Running go tool vet $(VETARGS) ."
|
||||||
|
@go tool vet $(VETARGS) . ; if [ $$? -eq 1 ]; then \
|
||||||
|
echo ""; \
|
||||||
|
echo "Vet found suspicious constructs. Please check the reported constructs"; \
|
||||||
|
echo "and fix them if necessary before submitting the code for reviewal."; \
|
||||||
|
fi
|
||||||
|
|
||||||
web:
|
web:
|
||||||
./scripts/website_run.sh
|
./scripts/website_run.sh
|
||||||
|
|
||||||
web-push:
|
web-push:
|
||||||
./scripts/website_push.sh
|
./scripts/website_push.sh
|
||||||
|
|
||||||
.PHONY: all cov deps integ test web web-push
|
.PHONY: all cov deps integ test vet web web-push
|
||||||
|
|
Loading…
Reference in New Issue