Skip to content
This repository was archived by the owner on May 17, 2024. It is now read-only.

Commit 9d9d164

Browse files
committed
Update verify to use go mod verify and fix linter
Signed-off-by: JoshVanL <[email protected]>
1 parent 12aa407 commit 9d9d164

36 files changed

+10186
-20
lines changed

Makefile

Lines changed: 5 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -11,17 +11,13 @@ help: ## display this help
1111
UNAME_S := $(shell uname -s)
1212
ifeq ($(UNAME_S),Linux)
1313
SHASUM := sha256sum -c
14-
DEP_URL := https://github.com/golang/dep/releases/download/v0.5.1/dep-linux-amd64
15-
DEP_HASH := 7479cca72da0596bb3c23094d363ea32b7336daa5473fa785a2099be28ecd0e3
1614
KUBECTL_URL := https://storage.googleapis.com/kubernetes-release/release/v1.13.3/bin/linux/amd64/kubectl
1715
KUBECTL_HASH := f3be209a48394e0e649b30ea376ce5093205fd6769c12e62c7ab39a0827c26fb
1816
GOLANGCILINT_URL := https://github.com/golangci/golangci-lint/releases/download/v1.15.0/golangci-lint-1.15.0-linux-amd64.tar.gz
1917
GOLANGCILINT_HASH := f37f4a15eb309578b0546703da5ea96bc5bd472f45f204338051aaca6fbbfc5b
2018
endif
2119
ifeq ($(UNAME_S),Darwin)
2220
SHASUM := shasum -a 256 -c
23-
DEP_URL := https://github.com/golang/dep/releases/download/v0.5.1/dep-darwin-amd64
24-
DEP_HASH := 7479cca72da0596bb3c23094d363ea32b7336daa5473fa785a2099be28ecd0e3
2521
KUBECTL_URL := https://storage.googleapis.com/kubernetes-release/release/v1.13.3/bin/darwin/amd64/kubectl
2622
KUBECTL_HASH := 2ff06345a02636f1e6934f19dbc55452b587e06b2828c775dcdb29229c8da40f
2723
GOLANGCILINT_URL := https://github.com/golangci/golangci-lint/releases/download/v1.15.0/golangci-lint-1.15.0-darwin-amd64.tar.gz
@@ -32,12 +28,6 @@ $(BINDIR)/mockgen:
3228
mkdir -p $(BINDIR)
3329
go build -o $(BINDIR)/mockgen ./vendor/github.com/golang/mock/mockgen
3430

35-
$(BINDIR)/dep:
36-
mkdir -p $(BINDIR)
37-
curl -sL -o $@ $(DEP_URL)
38-
echo "$(DEP_HASH) $@" | $(SHASUM)
39-
chmod +x $@
40-
4131
$(BINDIR)/kubectl:
4232
mkdir -p $(BINDIR)
4333
curl --fail -sL -o $(BINDIR)/.kubectl $(KUBECTL_URL)
@@ -53,13 +43,13 @@ $(BINDIR)/golangci-lint:
5343
mv $(BINDIR)/.golangci-lint/*/golangci-lint $(BINDIR)/golangci-lint
5444
rm -rf $(BINDIR)/.golangci-lint $(BINDIR)/.golangci-lint.tar.gz
5545

56-
depend: $(BINDIR)/mockgen $(BINDIR)/dep $(BINDIR)/kubectl $(BINDIR)/golangci-lint
46+
depend: $(BINDIR)/mockgen $(BINDIR)/kubectl $(BINDIR)/golangci-lint
5747

5848
verify_boilerplate:
5949
$(HACK_DIR)/verify-boilerplate.sh
6050

61-
verify_vendor: $(BINDIR)/dep
62-
$(BINDIR)/dep ensure -no-vendor -dry-run -v
51+
verify_vendor:
52+
go mod verify
6353

6454
go_fmt:
6555
@set -e; \
@@ -73,8 +63,9 @@ go_fmt:
7363
go_vet:
7464
go vet $$(go list ./pkg/... ./cmd/...)
7565

66+
# We have to make sure we omit ./hack/tools
7667
go_lint: $(BINDIR)/golangci-lint ## lint golang code for problems
77-
$(BINDIR)/golangci-lint run
68+
go list -f '{{.Dir}}' ./... | fgrep -v hack/tools | xargs realpath --relative-to=. | xargs $(BINDIR)/golangci-lint run
7869

7970
clean: ## clean up created files
8071
rm -rf \

go.mod

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ go 1.12
44

55
require (
66
github.com/Masterminds/semver v1.4.2
7-
github.com/golang/mock v1.3.1
7+
github.com/golang/mock v0.0.0-20160127222235-bd3c8e81be01
88
github.com/heptiolabs/healthcheck v0.0.0-20180807145615-6ff867650f40
99
github.com/spf13/cobra v0.0.5
1010
gopkg.in/square/go-jose.v2 v2.3.1
@@ -45,6 +45,8 @@ replace (
4545
sigs.k8s.io/structured-merge-diff => sigs.k8s.io/structured-merge-diff v0.0.0-20190302045857-e85c7b244fd2
4646
)
4747

48-
replace sigs.k8s.io/kind => sigs.k8s.io/kind v0.4.0
49-
50-
replace k8s.io/kubernetes => k8s.io/kubernetes v1.15.0
48+
replace (
49+
github.com/golang/mock => github.com/golang/mock v1.3.1
50+
k8s.io/kubernetes => k8s.io/kubernetes v1.15.0
51+
sigs.k8s.io/kind => sigs.k8s.io/kind v0.4.0
52+
)

go.sum

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,6 @@ github.com/golang/glog v0.0.0-20160126235308-23def4e6c14b h1:VKtxabqXZkF25pY9ekf
153153
github.com/golang/glog v0.0.0-20160126235308-23def4e6c14b/go.mod h1:SBH7ygxi8pfUlaOkMMuAQtPIUF8ecWP5IEl/CR7VP2Q=
154154
github.com/golang/groupcache v0.0.0-20160516000752-02826c3e7903 h1:LbsanbbD6LieFkXbj9YNNBupiGHJgFeLpO0j0Fza1h8=
155155
github.com/golang/groupcache v0.0.0-20160516000752-02826c3e7903/go.mod h1:cIg4eruTrX1D+g88fzRXU5OdNfaM+9IcxsU14FzY7Hc=
156-
github.com/golang/mock v0.0.0-20160127222235-bd3c8e81be01/go.mod h1:oTYuIxOrZwtPieC+H1uAHpcLFnEyAGVDL/k47Jfbm0A=
157156
github.com/golang/mock v1.3.1 h1:qGJ6qTW+x6xX/my+8YUVl4WNpX9B7+/l2tRsHGZ7f2s=
158157
github.com/golang/mock v1.3.1/go.mod h1:sBzyDLLjw3U8JLTeZvSv8jJB+tU5PVekmnlKIyFUx0Y=
159158
github.com/golang/protobuf v1.2.0 h1:P3YflyNX/ehuJFLhxviNdFxQPkGK5cDcApsge1SqnvM=
@@ -379,6 +378,7 @@ golang.org/x/oauth2 v0.0.0-20190402181905-9f3314589c9a h1:tImsplftrFpALCYumobsd0
379378
golang.org/x/oauth2 v0.0.0-20190402181905-9f3314589c9a/go.mod h1:gOpvHmFTYa4IltrdGE7lF6nIHvwfUNPOp7c8zoXwtLw=
380379
golang.org/x/sync v0.0.0-20180314180146-1d60e4601c6f/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
381380
golang.org/x/sync v0.0.0-20181108010431-42b317875d0f/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
381+
golang.org/x/sync v0.0.0-20181221193216-37e7f081c4d4 h1:YUO/7uOKsKeq9UokNS62b8FYywz3ker1l1vDZRCRefw=
382382
golang.org/x/sync v0.0.0-20181221193216-37e7f081c4d4/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
383383
golang.org/x/sync v0.0.0-20190423024810-112230192c58 h1:8gQV6CLnAEikrhgkHFbMAEhagSSnXWGV915qUMm9mrU=
384384
golang.org/x/sync v0.0.0-20190423024810-112230192c58/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
@@ -399,6 +399,7 @@ golang.org/x/tools v0.0.0-20170824195420-5d2fd3ccab98/go.mod h1:n7NCudcB/nEzxVGm
399399
golang.org/x/tools v0.0.0-20180221164845-07fd8470d635/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ=
400400
golang.org/x/tools v0.0.0-20180917221912-90fa682c2a6e/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ=
401401
golang.org/x/tools v0.0.0-20190206041539-40960b6deb8e/go.mod h1:n7NCudcB/nEzxVGmLbDWY5pfWTLqBcC2KZ6jyYvM4mQ=
402+
golang.org/x/tools v0.0.0-20190328211700-ab21143f2384 h1:TFlARGu6Czu1z7q93HTxcP1P+/ZFC/IKythI5RzrnRg=
402403
golang.org/x/tools v0.0.0-20190328211700-ab21143f2384/go.mod h1:LCzVGOaR6xXOjkQ3onu1FJEFr0SW1gC7cKk1uF8kGRs=
403404
golang.org/x/tools v0.0.0-20190425150028-36563e24a262 h1:qsl9y/CJx34tuA7QCPNp86JNJe4spst6Ff8MjvPUdPg=
404405
golang.org/x/tools v0.0.0-20190425150028-36563e24a262/go.mod h1:RgjU9mgBXZiqYHBnxXauZ1Gv1EHHAz9KjViQ78xBX0Q=

hack/tools/tools.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
// Copyright Jetstack Ltd. See LICENSE for details.
2+
package tools
3+
4+
//+build tools
5+
6+
import (
7+
_ "github.com/golang/mock/mockgen"
8+
)

0 commit comments

Comments
 (0)