From b3a1a9a8b86c7627cea5701538daa47830e00a67 Mon Sep 17 00:00:00 2001
From: Bobby DeSimone <bobbydesimone@gmail.com>
Date: Thu, 12 Sep 2019 16:02:16 -0700
Subject: [PATCH] deployment: staple linter version

---
 .golangci.yml |  3 ++-
 Makefile      | 15 ++++++++++-----
 2 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/.golangci.yml b/.golangci.yml
index 40dc39eb5..a7a567a97 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -1,7 +1,7 @@
 # forked from istio
 service:
   # When updating this, also update bin/linters.sh accordingly
-  golangci-lint-version: 1.17.x # use the fixed version to not introduce new linters unexpectedly
+  golangci-lint-version: 1.18.x # use the fixed version to not introduce new linters unexpectedly
 run:
   # timeout for analysis, e.g. 30s, 5m, default is 1m
   deadline: 5m
@@ -37,6 +37,7 @@ linters:
     - scopelint
     - maligned
     - interfacer
+    - funlen
   fast: false
 
 linters-settings:
diff --git a/Makefile b/Makefile
index a7885974f..b6ea86756 100644
--- a/Makefile
+++ b/Makefile
@@ -27,10 +27,17 @@ CTIMEVAR=-X $(PKG)/internal/version.GitCommit=$(GITCOMMIT) \
 	-X $(PKG)/internal/version.ProjectURL=$(PKG)
 GO_LDFLAGS=-ldflags "-s -w $(CTIMEVAR)"
 GOOSARCHES = linux/amd64 darwin/amd64 windows/amd64
-
-
+GOLANGCI_VERSION = v1.18.0 # .... for some reason v1.18.0 misses?
+ 
 .PHONY: all
-all: clean lint spellcheck test build ## Runs a clean, build, fmt, lint, test, and vet.
+all: clean build-deps test lint spellcheck build ## Runs a clean, build, fmt, lint, test, and vet.
+
+
+.PHONY: build-deps
+build-deps: ## Install build dependencies
+	@echo "==> $@"
+	@cd /tmp; GO111MODULE=on go get -u github.com/client9/misspell/cmd/misspell
+	@cd /tmp; GO111MODULE=on go get github.com/golangci/golangci-lint/cmd/golangci-lint@${GOLANGCI_VERSION}
 
 .PHONY: docs
 docs: ## Start the vuepress docs development server
@@ -50,7 +57,6 @@ build: ## Builds dynamic executables and/or packages.
 .PHONY: lint
 lint: ## Verifies `golint` passes.
 	@echo "==> $@"
-	@GO111MODULE=off go get -u github.com/golangci/golangci-lint/cmd/golangci-lint
 	@golangci-lint run ./...
 
 .PHONY: test
@@ -61,7 +67,6 @@ test: ## Runs the go tests.
 .PHONY: spellcheck
 spellcheck: # Spellcheck docs
 	@echo "==> Spell checking docs..."
-	@GO111MODULE=off go get -u github.com/client9/misspell/cmd/misspell
 	@misspell -error -source=text docs/