From dff606cd86e43586d30e5c171f9366ef6e8646ae Mon Sep 17 00:00:00 2001
From: Jonathan Yu <jonathan@coder.com>
Date: Fri, 7 May 2021 16:02:44 +0000
Subject: [PATCH] update golangci-lint configuration

* Update golangci-lint from v1.26 to v1.39
* Update golangci-lint-action from v0.2.0 to v2
* Avoid using enable-all per golangci-lint's recommendation, and
  instead enable all linters
---
 .github/workflows/golangci-lint.yml |  4 +--
 .golangci.yml                       | 44 +++++++++++++++++++++++++----
 2 files changed, 41 insertions(+), 7 deletions(-)

diff --git a/.github/workflows/golangci-lint.yml b/.github/workflows/golangci-lint.yml
index 0a579a1..156e791 100644
--- a/.github/workflows/golangci-lint.yml
+++ b/.github/workflows/golangci-lint.yml
@@ -13,6 +13,6 @@ jobs:
     steps:
       - uses: actions/checkout@v2
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v0.2.0
+        uses: golangci/golangci-lint-action@v2
         with:
-          version: v1.26
+          version: v1.39
diff --git a/.golangci.yml b/.golangci.yml
index 9c19e63..72e618e 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -1,6 +1,40 @@
 linters:
-  enable-all: true
-  disable:
-  - funlen
-  - gochecknoglobals
-  - lll
+  disable-all: true
+  enable:
+    - bodyclose
+    - deadcode
+    - depguard
+    - dogsled
+    - dupl
+    - errcheck
+    - exportloopref
+    - exhaustive
+    - funlen
+    - gochecknoinits
+    - goconst
+    - gocritic
+    - gocyclo
+    - gofmt
+    - goimports
+    - golint
+    - gomnd
+    - goprintffuncname
+    - gosec
+    - gosimple
+    - govet
+    - ineffassign
+    - lll
+    - misspell
+    - nakedret
+    - noctx
+    - nolintlint
+    - rowserrcheck
+    - staticcheck
+    - structcheck
+    - stylecheck
+    # - typecheck
+    - unconvert
+    - unparam
+    - unused
+    - varcheck
+    - whitespace