ci(lint): fix linting issue by bumping the linter to v1.56.2

openshift-online · Feb 29, 2024 · 00ccc34 · 00ccc34
1 parent 8a0cc00
commit 00ccc34
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/.github/workflows/check-pull-request.yaml b/.github/workflows/check-pull-request.yaml
@@ -59,7 +59,7 @@ jobs:
     - name: Run the linter
       uses: golangci/golangci-lint-action@v3
       with:
-        version: v1.51.0
+        version: v1.56.2
         args: --timeout 10m
         skip-pkg-cache: true
         skip-build-cache: true

diff --git a/authentication/transport_wrapper.go b/authentication/transport_wrapper.go
@@ -516,7 +516,7 @@ func (b *TransportWrapperBuilder) Build(ctx context.Context) (result *TransportW
 			registered, ok := err.(prometheus.AlreadyRegisteredError)
 			if ok {
 				tokenCountMetric = registered.ExistingCollector.(*prometheus.CounterVec)
-				err = nil
+				err = nil //nolint:all
 			} else {
 				return
 			}

diff --git a/metrics/handler_wrapper.go b/metrics/handler_wrapper.go
@@ -169,7 +169,7 @@ func (b *HandlerWrapperBuilder) Build() (result *HandlerWrapper, err error) {
 		registered, ok := err.(prometheus.AlreadyRegisteredError)
 		if ok {
 			requestCount = registered.ExistingCollector.(*prometheus.CounterVec)
-			err = nil
+			err = nil //nolint:all
 		} else {
 			return
 		}

diff --git a/metrics/transport_wrapper.go b/metrics/transport_wrapper.go
@@ -160,7 +160,7 @@ func (b *TransportWrapperBuilder) Build() (result *TransportWrapper, err error)
 		registered, ok := err.(prometheus.AlreadyRegisteredError)
 		if ok {
 			requestCount = registered.ExistingCollector.(*prometheus.CounterVec)
-			err = nil
+			err = nil //nolint:all
 		} else {
 			return
 		}