Skip to content
This repository was archived by the owner on Nov 7, 2022. It is now read-only.

Commit 2b64001

Browse files
author
Paulo Janotti
authored
Add go-cmp to install-tools target (#528)
Use the general install-tools target so no one gets the vet failure if they follow the CONTRIBUTING.md instructions. Currently first attempt will fail with the download of go-cmp.
1 parent 73558e6 commit 2b64001

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

Makefile

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ test:
3838
$(GOTEST) $(GOTEST_OPT) $(ALL_PKGS)
3939

4040
.PHONY: travis-ci
41-
travis-ci: fmt install-go-cmp vet lint test-with-cover
41+
travis-ci: fmt vet lint test-with-cover
4242

4343
.PHONY: test-with-cover
4444
test-with-cover:
@@ -85,9 +85,6 @@ vet:
8585
.PHONY: install-tools
8686
install-tools:
8787
go get golang.org/x/lint/golint
88-
89-
.PHONY: install-go-cmp
90-
install-go-cmp:
9188
go get -u github.com/google/go-cmp/cmp
9289

9390
.PHONY: agent

0 commit comments

Comments
 (0)