.github: Bump golangci/golangci-lint-action from 6 to 7 (#24)

* .github: Bump golangci/golangci-lint-action from 6 to 7

Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 6 to 7.
- [Release notes](https://github.com/golangci/golangci-lint-action/releases)
- [Commits](golangci/golangci-lint-action@v6...v7)

---
updated-dependencies:
- dependency-name: golangci/golangci-lint-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

* update golangci-lint version

* fix linter warning

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Lovro Mažgon <[email protected]>

Author: dependabot[bot]

.github/workflows/lint.yml

@@ -28,8 +28,8 @@ jobs:
           go-version-file: 'go.mod'
 
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v6
+        uses: golangci/golangci-lint-action@v7
         with:
-          version: v1.52.2
+          version: v2.0.2
           working-directory: ${{ matrix.workdir }}
 

.golangci.yml

@@ -1,53 +1,63 @@
-linters-settings:
-  gofmt:
-    simplify: false
-  govet:
-    check-shadowing: false
-  nolintlint:
-    allow-unused: false # report any unused nolint directives
-    require-explanation: true # require an explanation for nolint directives
-    require-specific: true # require nolint directives to mention the specific linter being suppressed
-  gocyclo:
-    min-complexity: 20
-  goconst:
-    ignore-tests: true
-  goheader:
-    template-path: '.golangci.goheader.template'
-    values:
-      regexp:
-        copyright-year: 20[2-9]\d
-
+version: "2"
 linters:
-  disable-all: true
+  default: none
   enable:
     - bodyclose
     - dogsled
     - durationcheck
     - errcheck
     - errname
-    - exportloopref
     - goconst
     - gocritic
     - gocyclo
-    - gofmt
     - goheader
-    - goimports
-    - revive
     - gomoddirectives
     - gomodguard
     - goprintffuncname
     - gosec
-    - gosimple
     - govet
     - ineffassign
     - makezero
     - noctx
     - nolintlint
     - predeclared
+    - revive
     - staticcheck
-    - stylecheck
-    - typecheck
     - unconvert
     - unused
     - wastedassign
     - whitespace
+  settings:
+    gocyclo:
+      min-complexity: 20
+    goheader:
+      values:
+        regexp:
+          copyright-year: 20[2-9]\d
+      template-path: .golangci.goheader.template
+    nolintlint:
+      require-explanation: true
+      require-specific: true
+      allow-unused: false
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    rules:
+      - linters:
+          - goconst
+        path: (.+)_test\.go
+    paths:
+      - examples/.*
+formatters:
+  enable:
+    - gofmt
+    - goimports
+  settings:
+    gofmt:
+      simplify: false
+  exclusions:
+    generated: lax

bwgrpc/client_test.go

@@ -83,7 +83,7 @@ func startTestServer(t *testing.T, lis net.Listener, resp *testproto.TestRespons
 	mockServer.EXPECT().
 		TestRPC(gomock.Any(), gomock.Any()).
 		DoAndReturn(
-			func(ctx context.Context, request *testproto.TestRequest) (*testproto.TestResponse, error) {
+			func(context.Context, *testproto.TestRequest) (*testproto.TestResponse, error) {
 				return resp, nil
 			},
 		)