From 25827f01aa4547fc64e8bb453408651ee1572dbc Mon Sep 17 00:00:00 2001
From: Tyler Yahn <MrAlias@users.noreply.github.com>
Date: Mon, 14 Feb 2022 09:09:41 -0800
Subject: [PATCH] Lock down golangci-lint config (#2602)

Disable all default linters prior to enabling the ones we want to ensure
that no upgrade that include new default linters introduce changes to
the CI system.
---
 .golangci.yml | 19 +++++++++++++++++--
 1 file changed, 17 insertions(+), 2 deletions(-)

diff --git a/.golangci.yml b/.golangci.yml
index d40bdedc397..7a5fdc07abf 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -4,11 +4,26 @@ run:
   tests: true #Default
 
 linters:
+  # Disable everything by default so upgrades to not include new "default
+  # enabled" linters.
+  disable-all: true
+  # Specifically enable linters we want to use.
   enable:
-    - misspell
+    - deadcode
+    - errcheck
+    - gofmt
     - goimports
+    - gosimple
+    - govet
+    - ineffassign
+    - misspell
     - revive
-    - gofmt
+    - staticcheck
+    - structcheck
+    - typecheck
+    - unused
+    - varcheck
+
 
 issues:
   exclude-rules: