From b55c0620431b1a3c8fda3d327b425809c2613d73 Mon Sep 17 00:00:00 2001 From: Paddy Date: Thu, 14 Feb 2019 17:09:18 -0800 Subject: [PATCH] Switch to using golangci-lint instead of go-metalinter (#437) * Use golangci-lint instead of gometalinter. * Set go flags to vendor. --- .golangci.yml | 26 ++++++++++++++++++++++++++ .travis.yml | 4 ++-- GNUmakefile | 8 ++++---- 3 files changed, 32 insertions(+), 6 deletions(-) create mode 100644 .golangci.yml diff --git a/.golangci.yml b/.golangci.yml new file mode 100644 index 0000000000..433ae75563 --- /dev/null +++ b/.golangci.yml @@ -0,0 +1,26 @@ +issues: + max-per-linter: 0 + max-same-issues: 0 + +linters: + disable-all: true + enable: + - deadcode + - errcheck + - gofmt + - gosimple + - ineffassign + - misspell + - staticcheck + - structcheck + - unconvert + - unused + - varcheck + - vet + +linters-settings: + errcheck: + ignore: github.com/hashicorp/terraform/helper/schema:ForceNew|Set,fmt:.*,io:Close,github.com/terraform-providers/terraform-provider-google-beta/google-beta:Set + +run: + modules-download-mode: vendor diff --git a/.travis.yml b/.travis.yml index 8c29147b5f..1f406a9b57 100644 --- a/.travis.yml +++ b/.travis.yml @@ -6,7 +6,8 @@ language: go go: - "1.11.x" env: - - GO111MODULE=off + - GO111MODULE=on + GOFLAGS=-mod=vendor install: # This script is used by the Travis build to install a cookie for @@ -24,7 +25,6 @@ script: branches: only: - master - - 2.0.0 matrix: fast_finish: true allow_failures: diff --git a/GNUmakefile b/GNUmakefile index e7af6e548a..1469fa69d0 100644 --- a/GNUmakefile +++ b/GNUmakefile @@ -27,17 +27,17 @@ fmtcheck: lint: @echo "==> Checking source code against linters..." - @bash -c "GO111MODULE=off gometalinter -d ./... 2> >(egrep '(^DEBUG.*linter took|^DEBUG.*total elapsed|^[^D])' >&2)" + @golangci-lint run ./$(DIR_NAME) tools: @echo "==> installing required tooling..." - GO111MODULE=off go get -u github.com/alecthomas/gometalinter - GO111MODULE=off gometalinter --install + GO111MODULE=off go get -u github.com/client9/misspell/cmd/misspell + GO111MODULE=off go get -u github.com/golangci/golangci-lint/cmd/golangci-lint test-compile: @if [ "$(TEST)" = "./..." ]; then \ echo "ERROR: Set TEST to a specific package. For example,"; \ - echo " make test-compile TEST=./$(PKG_NAME)"; \ + echo " make test-compile TEST=./$(DIR_NAME)"; \ exit 1; \ fi go test -c $(TEST) $(TESTARGS)