From d5e49845f3555b7ef47ebdc1ec17101a8b4b74dc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Torkel=20=C3=96degaard?= <torkel@grafana.com>
Date: Mon, 21 Jan 2019 13:02:30 +0100
Subject: [PATCH] Moved gometalinter to a script instead of seperate commands
 in circleci file, removed megacheck and added staticcheck

---
 .circleci/config.yml          | 15 ++-------------
 scripts/build/gometalinter.sh | 22 ++++++++++++++++++++++
 2 files changed, 24 insertions(+), 13 deletions(-)
 create mode 100755 scripts/build/gometalinter.sh

diff --git a/.circleci/config.yml b/.circleci/config.yml
index 876db1d4823..008d2faf57c 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -81,20 +81,9 @@ jobs:
     working_directory: /go/src/github.com/grafana/grafana
     steps:
       - checkout
-      - run: 'go get -u github.com/alecthomas/gometalinter'
-      - run: 'go get -u github.com/tsenart/deadcode'
-      - run: 'go get -u github.com/jgautheron/goconst/cmd/goconst'
-      - run: 'go get -u github.com/gordonklaus/ineffassign'
-      - run: 'go get -u honnef.co/go/tools/cmd/megacheck'
-      - run: 'go get -u github.com/opennota/check/cmd/structcheck'
-      - run: 'go get -u github.com/mdempsky/unconvert'
-      - run: 'go get -u github.com/opennota/check/cmd/varcheck'
       - run:
-          name: run linters
-          command: 'gometalinter --enable-gc --vendor --deadline 10m --disable-all --enable=deadcode --enable=goconst --enable=gofmt --enable=ineffassign --enable=megacheck --enable=structcheck --enable=unconvert --enable=varcheck ./...'
-      - run:
-          name: run go vet
-          command: 'go vet ./pkg/...'
+          name: frontend tests
+          command: './scripts/build/gometalinter.sh'
 
   test-frontend:
     docker:
diff --git a/scripts/build/gometalinter.sh b/scripts/build/gometalinter.sh
new file mode 100755
index 00000000000..9c77220991f
--- /dev/null
+++ b/scripts/build/gometalinter.sh
@@ -0,0 +1,22 @@
+#!/bin/bash -e
+
+go get -u github.com/alecthomas/gometalinter
+go get -u github.com/tsenart/deadcode
+go get -u github.com/jgautheron/goconst/cmd/goconst
+go get -u github.com/gordonklaus/ineffassign
+go get -u github.com/opennota/check/cmd/structcheck
+go get -u github.com/mdempsky/unconvert
+go get -u github.com/opennota/check/cmd/varcheck
+go get -u honnef.co/go/tools/cmd/staticcheck
+
+gometalinter --enable-gc --vendor --deadline 10m --disable-all \
+  --enable=deadcode \
+  --enable=goconst \
+  --enable=gofmt \
+  --enable=ineffassign \
+  --enable=structcheck \
+  --enable=unconvert \
+  --enable=varcheck \
+  --enable=staticcheck
+
+go vet ./pkg/...