Migrate the golangci-lint config to v2

.github/workflows/pr.yml

@@ -26,9 +26,9 @@ jobs:
         if [[ -n $(git status --porcelain) ]]; then echo "git repo is dirty after running go generate -- please don't modify generated files"; echo $(git diff);echo $(git status --porcelain); exit 1; fi
 
     - name: golangci-lint
-      uses: golangci/golangci-lint-action@v6
+      uses: golangci/golangci-lint-action@v8
       with:
-          version: v1.60
+          version: v2.1.6
           args: -v --timeout=5m
 
   test:

.github/workflows/push.yml

@@ -29,10 +29,10 @@ jobs:
         if [[ -n $(git status --porcelain) ]]; then echo "git repo is dirty after running go generate -- please don't modify generated files"; echo $(git diff);echo $(git status --porcelain); exit 1; fi
 
     - name: golangci-lint
-      uses: golangci/golangci-lint-action@v6
+      uses: golangci/golangci-lint-action@v8
       with:
-          version: v1.60
-          args: -v --timeout=5m
+        version: v2.1.6
+        args: -v --timeout=5m
 
   test:
     strategy:

.golangci.yml

@@ -1,19 +1,41 @@
+version: "2"
+run:
+  issues-exit-code: 1
 linters:
-  disable-all: true
+  default: none
   enable:
-    - gofmt
     - staticcheck
     - gosec
-    - gosimple
     - govet
     - ineffassign
     - misspell
-
-linters-settings:
-  gosec:
-    excludes:
-      - G115 # Conversions from int -> uint etc.
-issues:
-  exclude-generated: disable
-run:
-  issues-exit-code: 1
+  settings:
+    staticcheck:
+      checks:
+        - all
+        - '-QF1003'
+        - '-QF1008'
+        - '-ST1003'
+    gosec:
+      excludes:
+        - G115
+  exclusions:
+    generated: disable
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    paths:
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
+  enable:
+    - gofmt
+  exclusions:
+    generated: disable
+    paths:
+      - third_party$
+      - builtin$
+      - examples$