Skip to content
This repository has been archived by the owner on Jan 22, 2021. It is now read-only.

A few vet and errchk issues that could be fixed #46

Open
tomdee opened this issue Jun 12, 2018 · 0 comments
Open

A few vet and errchk issues that could be fixed #46

tomdee opened this issue Jun 12, 2018 · 0 comments

Comments

@tomdee
Copy link
Contributor

tomdee commented Jun 12, 2018

docker run --rm \
	-e LOCAL_USER_ID=1001 \
	-v /home/tom/go/src/github.com/projectcalico/calico-bgp-daemon:/go/src/github.com/projectcalico/calico-bgp-daemon \
	calico/go-build:v0.16 sh -c '\
		cd  /go/src/github.com/projectcalico/calico-bgp-daemon && \
		gometalinter --deadline=300s --disable-all --enable=goimports --enable=vet --enable=errcheck --vendor -s gobgp ./...'
Starting with UID : 1001
ipam.go:176::error: unreachable code (vet)
main.go:382::error: missing argument for Errorf("%s"): format reads arg 1, have only 0 args (vet)
k8s.go:128:14:warning: error return value not checked (p.ipam.sync()) (errcheck)
k8s.go:129:26:warning: error return value not checked (p.k8scli.checkBGPConfig()) (errcheck)
main.go:154:19:warning: error return value not checked (netlink.RouteDel(&route)) (errcheck)
Makefile:162: recipe for target 'static-checks' failed
make: *** [static-checks] Error 1
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants