build: Give linters longer and cleanup goclean.sh.
This modifies the goclean.sh script to expand the deadline for the linters to run to 4 minutes and ensures the comments match reality.
This commit is contained in:
parent
34b9721494
commit
afec1bd124
1 changed files with 7 additions and 5 deletions
12
goclean.sh
12
goclean.sh
|
@ -3,11 +3,13 @@
|
||||||
# 1. gofmt (http://golang.org/cmd/gofmt/)
|
# 1. gofmt (http://golang.org/cmd/gofmt/)
|
||||||
# 2. golint (https://github.com/golang/lint)
|
# 2. golint (https://github.com/golang/lint)
|
||||||
# 3. go vet (http://golang.org/cmd/vet)
|
# 3. go vet (http://golang.org/cmd/vet)
|
||||||
# 4. race detector (http://blog.golang.org/race-detector)
|
# 4. gosimple (https://github.com/dominikh/go-simple)
|
||||||
# 5. test coverage (http://blog.golang.org/cover)
|
# 5. unconvert (https://github.com/mdempsky/unconvert)
|
||||||
|
# 6. race detector (http://blog.golang.org/race-detector)
|
||||||
|
# 7. test coverage (http://blog.golang.org/cover)
|
||||||
#
|
#
|
||||||
# gometaling (github.com/alecthomas/gometalinter) is used to run each each
|
# gometalinter (github.com/alecthomas/gometalinter) is used to run each static
|
||||||
# static checker.
|
# checker.
|
||||||
|
|
||||||
set -ex
|
set -ex
|
||||||
|
|
||||||
|
@ -18,7 +20,7 @@ test -z "$(gometalinter --disable-all \
|
||||||
--enable=vet \
|
--enable=vet \
|
||||||
--enable=gosimple \
|
--enable=gosimple \
|
||||||
--enable=unconvert \
|
--enable=unconvert \
|
||||||
--deadline=20s $(glide novendor) | grep -v 'ALL_CAPS\|OP_' 2>&1 | tee /dev/stderr)"
|
--deadline=4m $(glide novendor) | grep -v 'ALL_CAPS\|OP_' 2>&1 | tee /dev/stderr)"
|
||||||
env GORACE="halt_on_error=1" go test -v -race -tags rpctest $(glide novendor)
|
env GORACE="halt_on_error=1" go test -v -race -tags rpctest $(glide novendor)
|
||||||
|
|
||||||
# Run test coverage on each subdirectories and merge the coverage profile.
|
# Run test coverage on each subdirectories and merge the coverage profile.
|
||||||
|
|
Loading…
Add table
Reference in a new issue