ci: refactor linter configurations and update GitHub workflows

- Update golangci-lint-action to version 7 in GitHub workflow
- Specify version v2.0 for golangci-lint-action in GitHub workflow
- Set golangci.yml version to "2"
- Change linter configuration to "default: none" in golangci.yml
- Replace multiple linter settings with a single "settings" entry in golangci.yml
- Add exclusions and presets for generated code in golangci.yml
- Add paths exclusions for third_party, builtin, and examples in golangci.yml
- Enable gofmt, gofumpt, and goimports formatters in golangci.yml
- Replace ioutil with io in size_test.go

Signed-off-by: Bo-Yi Wu <[email protected]>

Author: appleboy

.github/workflows/go.yml

@@ -23,8 +23,9 @@ jobs:
           go-version-file: go.mod
           check-latest: true
       - name: Setup golangci-lint
-        uses: golangci/golangci-lint-action@v6
+        uses: golangci/golangci-lint-action@v7
         with:
+          version: v2.0
           args: --verbose
   test:
     strategy:

.golangci.yml

@@ -1,24 +1,19 @@
+version: "2"
 linters:
-  enable-all: false
-  disable-all: true
-  fast: false
+  default: none
   enable:
     - bodyclose
     - depguard
     - dogsled
     - dupl
     - errcheck
-    - exportloopref
     - exhaustive
     - gochecknoinits
     - goconst
     - gocritic
     - gocyclo
-    - gofmt
-    - goimports
     - goprintffuncname
     - gosec
-    - gosimple
     - govet
     - ineffassign
     - lll
@@ -28,31 +23,46 @@ linters:
     - nolintlint
     - rowserrcheck
     - staticcheck
-    - stylecheck
-    - typecheck
     - unconvert
     - unparam
     - unused
     - whitespace
+  settings:
+    depguard:
+      rules:
+        Main:
+          files:
+            - $all
+            - '!$test'
+          allow:
+            - $gostd
+            - github.com/gin-gonic/gin
+        Test:
+          files:
+            - $test
+          allow:
+            - $gostd
+            - github.com/gin-gonic/gin
+            - github.com/stretchr/testify
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    paths:
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
+  enable:
+    - gofmt
     - gofumpt
-
-linters-settings:
-  depguard:
-    rules:
-      Main:
-        files:
-          - $all
-          - "!$test"
-        allow:
-          - $gostd
-          - github.com/gin-gonic/gin
-      Test:
-        files:
-          - $test
-        allow:
-          - $gostd
-          - github.com/gin-gonic/gin
-          - github.com/stretchr/testify
-
-run:
-  timeout: 3m
+    - goimports
+  exclusions:
+    generated: lax
+    paths:
+      - third_party$
+      - builtin$
+      - examples$

size_test.go

@@ -2,7 +2,7 @@ package limits
 
 import (
 	"bytes"
-	"io/ioutil"
+	"io"
 	"net/http"
 	"net/http/httptest"
 	"testing"
@@ -14,7 +14,7 @@ func TestRequestSizeLimiterOK(t *testing.T) {
 	router := gin.New()
 	router.Use(RequestSizeLimiter(10))
 	router.POST("/test_ok", func(c *gin.Context) {
-		_, _ = ioutil.ReadAll(c.Request.Body)
+		_, _ = io.ReadAll(c.Request.Body)
 		if len(c.Errors) > 0 {
 			return
 		}
@@ -32,7 +32,7 @@ func TestRequestSizeLimiterOver(t *testing.T) {
 	router := gin.New()
 	router.Use(RequestSizeLimiter(10))
 	router.POST("/test_large", func(c *gin.Context) {
-		_, _ = ioutil.ReadAll(c.Request.Body)
+		_, _ = io.ReadAll(c.Request.Body)
 		if len(c.Errors) > 0 {
 			return
 		}