chore: update CI and configuration

Author: ldez

.github/workflows/pr.yml

@@ -28,26 +28,27 @@ jobs:
           git diff --exit-code go.mod
           git diff --exit-code go.sum
 
+  # TODO(ldez): must add uncommented when [email protected] (with golangci-lint v2 support) will be created.
   # We already run the current golangci-lint in tests, but here we test
   # our GitHub action with the latest stable golangci-lint.
-  golangci-lint:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-go@v5
-        with:
-          # https://github.com/actions/setup-go#supported-version-syntax
-          # ex:
-          # - 1.18beta1 -> 1.18.0-beta.1
-          # - 1.18rc1 -> 1.18.0-rc.1
-          go-version: ${{ env.GO_VERSION }}
-      - name: lint
-        uses: golangci/[email protected]
-        with:
-          version: latest
+#  golangci-lint:
+#    runs-on: ubuntu-latest
+#    steps:
+#      - uses: actions/checkout@v4
+#      - uses: actions/setup-go@v5
+#        with:
+#          # https://github.com/actions/setup-go#supported-version-syntax
+#          # ex:
+#          # - 1.18beta1 -> 1.18.0-beta.1
+#          # - 1.18rc1 -> 1.18.0-rc.1
+#          go-version: ${{ env.GO_VERSION }}
+#      - name: lint
+#        uses: golangci/[email protected]
+#        with:
+#          version: latest
 
   tests-on-windows:
-    needs: golangci-lint # run after golangci-lint action to not produce duplicated errors
+#    needs: golangci-lint ## TODO(ldez) disable during v2 developpment. # run after golangci-lint action to not produce duplicated errors
     runs-on: windows-latest
     steps:
       - uses: actions/checkout@v4
@@ -58,7 +59,7 @@ jobs:
         run: make.exe test
 
   tests-on-macos:
-    needs: golangci-lint # run after golangci-lint action to not produce duplicated errors
+#    needs: golangci-lint ## TODO(ldez) disable during v2 developpment. # run after golangci-lint action to not produce duplicated errors
     runs-on: macos-latest
     steps:
       - uses: actions/checkout@v4
@@ -69,7 +70,7 @@ jobs:
         run: make test
 
   tests-on-unix:
-    needs: golangci-lint # run after golangci-lint action to not produce duplicated errors
+#    needs: golangci-lint ## TODO(ldez) disable during v2 developpment. # run after golangci-lint action to not produce duplicated errors
     strategy:
       matrix:
         os:
@@ -116,7 +117,7 @@ jobs:
 
   # Note: the command `run` is tested by the previous steps (`make test`).
   commands:
-    needs: golangci-lint
+#    needs: golangci-lint ## TODO(ldez) disable during v2 development.
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4

.golangci.yml

@@ -11,6 +11,8 @@
 #
 # See the file `.golangci.reference.yml` to have a list of all available configuration options.
 
+version: "2"
+
 linters:
   disable-all: true
   # This list of linters is not a recommendation (same thing for all this configuration file).
@@ -31,8 +33,6 @@ linters:
     - gocritic
     - gocyclo
     - godox
-    - gofmt
-    - goimports
     - mnd
     - goprintffuncname
     - gosec
@@ -53,7 +53,63 @@ linters:
     - unparam
     - unused
     - whitespace
+  exclusions:
+    paths:
+      - test/testdata_etc # test files
+      - internal/go # extracted from Go code
+      - internal/x # extracted from x/tools code
+      - pkg/goformatters/gci/internal # extracted from gci code
+      - pkg/goanalysis/runner_checker.go # extracted from x/tools code
+    rules:
+      - path: (.+)_test\.go
+        linters:
+          - dupl
+          - mnd
+          - lll
+
+      # Deprecated linter options.
+      - path: pkg/golinters/errcheck/errcheck.go
+        linters: [staticcheck]
+        text: "SA1019: errCfg.Exclude is deprecated: use ExcludeFunctions instead"
+      - path: pkg/golinters/errcheck/errcheck.go
+        linters: [staticcheck]
+        text: "SA1019: errCfg.Ignore is deprecated: use ExcludeFunctions instead"
+      - path: pkg/golinters/govet/govet.go
+        linters: [staticcheck]
+        text: "SA1019: cfg.CheckShadowing is deprecated: the linter should be enabled inside Enable."
+      - path: pkg/golinters/godot/godot.go
+        linters: [staticcheck]
+        text: "SA1019: settings.CheckAll is deprecated: use Scope instead"
+      - path: pkg/goformatters/gci/gci.go
+        linters: [staticcheck]
+        text: "SA1019: settings.LocalPrefixes is deprecated: use Sections instead."
 
+      # Related to `run.go`, it cannot be removed.
+      - path: pkg/golinters/gofumpt/gofumpt.go
+        linters: [staticcheck]
+        text: "SA1019: settings.LangVersion is deprecated: use the global `run.go` instead."
+
+      # Based on existing code, the modifications should be limited to make maintenance easier.
+      - path: pkg/golinters/unused/unused.go
+        linters: [gocritic]
+        text: "rangeValCopy: each iteration copies 160 bytes \\(consider pointers or indexing\\)"
+
+      # Related to the result of computation but divided multiple times by 1024.
+      - path: test/bench/bench_test.go
+        linters: [gosec]
+        text: "G115: integer overflow conversion uint64 -> int"
+
+formatters:
+  enable:
+    - gofmt
+    - goimports
+  settings:
+    gofmt:
+      rewrite-rules:
+        - pattern: 'interface{}'
+          replacement: 'any'
+    goimports:
+      local-prefixes: github.com/golangci/golangci-lint
 
 linters-settings:
   depguard:
@@ -95,12 +151,6 @@ linters-settings:
   godox:
     keywords:
       - FIXME
-  gofmt:
-    rewrite-rules:
-      - pattern: 'interface{}'
-        replacement: 'any'
-  goimports:
-    local-prefixes: github.com/golangci/golangci-lint
   mnd:
     # don't include the "operation" and "assign"
     checks:
@@ -146,53 +196,5 @@ linters-settings:
       - name: unused-parameter
       - name: unused-receiver
 
-issues:
-  exclude-rules:
-    - path: (.+)_test\.go
-      linters:
-        - dupl
-        - mnd
-        - lll
-
-    # Deprecated linter options.
-    - path: pkg/golinters/errcheck/errcheck.go
-      linters: [staticcheck]
-      text: "SA1019: errCfg.Exclude is deprecated: use ExcludeFunctions instead"
-    - path: pkg/golinters/errcheck/errcheck.go
-      linters: [staticcheck]
-      text: "SA1019: errCfg.Ignore is deprecated: use ExcludeFunctions instead"
-    - path: pkg/golinters/govet/govet.go
-      linters: [staticcheck]
-      text: "SA1019: cfg.CheckShadowing is deprecated: the linter should be enabled inside Enable."
-    - path: pkg/golinters/godot/godot.go
-      linters: [staticcheck]
-      text: "SA1019: settings.CheckAll is deprecated: use Scope instead"
-    - path: pkg/goformatters/gci/gci.go
-      linters: [staticcheck]
-      text: "SA1019: settings.LocalPrefixes is deprecated: use Sections instead."
-
-    # Related to `run.go`, it cannot be removed.
-    - path: pkg/golinters/gofumpt/gofumpt.go
-      linters: [staticcheck]
-      text: "SA1019: settings.LangVersion is deprecated: use the global `run.go` instead."
-
-    # Based on existing code, the modifications should be limited to make maintenance easier.
-    - path: pkg/golinters/unused/unused.go
-      linters: [gocritic]
-      text: "rangeValCopy: each iteration copies 160 bytes \\(consider pointers or indexing\\)"
-
-    # Related to the result of computation but divided multiple times by 1024.
-    - path: test/bench/bench_test.go
-      linters: [gosec]
-      text: "G115: integer overflow conversion uint64 -> int"
-
-  exclude-dirs:
-    - test/testdata_etc # test files
-    - internal/go # extracted from Go code
-    - internal/x # extracted from x/tools code
-    - pkg/goformatters/gci/internal # extracted from gci code
-  exclude-files:
-    - pkg/goanalysis/runner_checker.go # extracted from x/tools code
-
 run:
   timeout: 5m