ci: update golangci-lint

Author: sichan-vonage

.github/workflows/golangci-lint.yml

@@ -20,14 +20,14 @@ jobs:
       - uses: actions/setup-go@v5
         with:
           go-version: '1.24.0'
-          cache: false
+          cache: true
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v3
+        uses: golangci/golangci-lint-action@v8
         with:
           # Require: The version of golangci-lint to use.
           # When `install-mode` is `binary` (default) the value can be v1.2 or v1.2.3 or `latest` to use the latest version.
           # When `install-mode` is `goinstall` the value can be v1.2.3, `latest`, or the hash of a commit.
-          version: v1.64.8
+          version: v2.6.2
 
           # Optional: working directory, useful for monorepos
           # working-directory: somedir

.golangci.yml

@@ -1,12 +1,4 @@
-linters-settings:
-  gofmt:
-    simplify: true
-  goimports:
-    local-prefixes: github.com/Vonage/vonage-cloud-runtime-cli
-  gocyclo:
-    min-complexity: 20
-  cyclop:
-    max-complexity: 20
+version: 2
 
 run:
   timeout: 10m
@@ -18,75 +10,66 @@ run:
     - node_modules
 
 linters:
-  enable-all: true
-  disable:
-    - gochecknoglobals
-    - gochecknoinits
-    - funlen
-    - wsl
-    - lll
-    - exhaustivestruct
-    - wrapcheck
-    - exhaustruct
-    - forbidigo
-    - godot
-    - gofumpt
-    - depguard
-    - goerr113
-    - ifshort
-    - interfacebloat
-    - ireturn
-    - paralleltest
-    - testpackage
-    - varnamelen
-    - tagliatelle
-    - nlreturn
-    - noctx
-    - gomnd
-    - forcetypeassert
-    - whitespace
-    - gci
-    - scopelint
-    - godox
-    - gosec
-    - nestif
-    - mirror
-    - thelper
-    - usestdlibvars
-    - nosnakecase
-    - contextcheck
-    - maligned
-    - unconvert
-    - durationcheck
-    - usestdlibvars
-    - thelper
-    - predeclared
-    - maintidx
-    - unparam
-    - perfsprint
-    - gosmopolitan
+  enable:
+    - errcheck
+    - govet
+    - staticcheck
+    - unused
+    - ineffassign
+    - misspell
+    - goconst
+    - gocyclo
+    - cyclop
+    - errname
+    - errorlint
+    - gocritic
+    - goprintffuncname
+    - nilerr
+    - rowserrcheck
+    - sqlclosecheck
+  settings:
+    gocyclo:
+      min-complexity: 25
+    cyclop:
+      max-complexity: 25
+      skip-tests: true
+
+  exclusions:
+    rules:
+      - text: "Error return value of `.*Close` is not checked"
+        linters:
+          - errcheck
+      - text: "Error return value of `fmt.Fprint` is not checked"
+        linters:
+          - errcheck
+      - text: "Error return value of `fmt.Fprintf` is not checked"
+        linters:
+          - errcheck
+      - text: "Error return value of `fmt.Fprintln` is not checked"
+        linters:
+          - errcheck
+      - text: "don't use an underscore in package name"
+        linters:
+          - golint
+      - text: "package comment should be of the form"
+        linters:
+          - golint
+      - path: _test\.go
+        linters:
+          - funlen
+          - goconst
+          - dupl
+          - gocognit
+          - testifylint
+          - cyclop
+          - gocyclo
+          - errcheck
+      - path: tests/integration
+        linters:
+          - funlen
+          - goconst
+          - dupl
+          - cyclop
+          - gocyclo     
+
 
-issues:
-  exclude-rules:
-    - text: "don't use an underscore in package name"
-      linters:
-        - golint
-    - text: "package comment should be of the form"
-      linters:
-        - golint
-    - path: (.+)_test.go
-      linters:
-        - funlen
-        - goconst
-        - dupl
-        - exhaustivestruct
-        - gocognit
-        - structcheck
-        - testifylint
-    - path: tests/integration
-      linters:
-        - funlen
-        - goconst
-        - dupl
-        - exhaustivestruct
-        - structcheck