Replace gometalinter with GolangCI-Lint

Running test.sh is down from 34s to 20s on my
machine.

Also, from now on, run go lint.

Signed-off-by: David Gageot <[email protected]>

Author: dgageot

Diff for: hack/gometalinter.json

@@ -16,18 +16,22 @@
 
 set -e -o pipefail
 
-SCRIPTDIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
-
-install_gometalinter() {
-	echo "Installing gometalinter.v2"
-	go get -u gopkg.in/alecthomas/gometalinter.v2
-	gometalinter.v2 --install
-}
-
-if ! [ -x "$(command -v gometalinter.v2)" ]; then
-  install_gometalinter
+if ! [ -x "$(command -v golangci-lint)" ]; then
+	echo "Installing GolangCI-Lint"
+	go get -u github.com/golangci/golangci-lint/cmd/golangci-lint
 fi
 
-gometalinter.v2 \
-	--deadline 5m \
-	--config $SCRIPTDIR/gometalinter.json ./...
+golangci-lint run \
+	--no-config \
+	-E goimports \
+	-E interfacer \
+	-E unconvert \
+	-E goconst \
+	-E maligned \
+	-D errcheck
+
+# From now on, run go lint.
+golangci-lint run \
+	--disable-all \
+	-E golint \
+	--new-from-rev bed41e9a77431990cc8504c0955252c851934b89

Diff for: hack/gometalinter.sh renamed to hack/linter.sh

@@ -60,7 +60,6 @@ func NewLogAggregator(out io.Writer, podSelector PodSelector, colorPicker ColorP
 }
 
 func (a *LogAggregator) Start(ctx context.Context) error {
-	// Start logs
 	kubeclient, err := Client()
 	if err != nil {
 		return errors.Wrap(err, "getting k8s client")
@@ -101,8 +100,7 @@ func (a *LogAggregator) Start(ctx context.Context) error {
 	return nil
 }
 
-// nolint: interfacer
-func (a *LogAggregator) streamLogs(ctx context.Context, client corev1.CoreV1Interface, pod *v1.Pod) error {
+func (a *LogAggregator) streamLogs(ctx context.Context, client corev1.PodsGetter, pod *v1.Pod) error {
 	pods := client.Pods(pod.Namespace)
 	if err := WaitForPodReady(pods, pod.Name); err != nil {
 		return errors.Wrap(err, "waiting for pod ready")

Diff for: pkg/skaffold/kubernetes/log.go

@@ -116,8 +116,7 @@ func (s *PodStore) Stop() {
 	close(s.stopCh)
 }
 
-// nolint: interfacer
-func NewPodStore(c kubernetes.Interface, namespace string, label labels.Selector, field fields.Selector) *PodStore {
+func NewPodStore(c kubernetes.Interface, namespace string, label fmt.Stringer, field fmt.Stringer) *PodStore {
 	lw := &cache.ListWatch{
 		ListFunc: func(options meta_v1.ListOptions) (runtime.Object, error) {
 			options.LabelSelector = label.String()

Diff for: pkg/skaffold/kubernetes/wait.go

@@ -31,7 +31,7 @@ echo "Running validation scripts..."
 scripts=(
     "hack/boilerplate.sh"
     "hack/gofmt.sh"
-    "hack/gometalinter.sh"
+    "hack/linter.sh"
     "hack/dep.sh"
 )
 fail=0