From 2d468ad289826cdf46af16a59f86f04e70403e5e Mon Sep 17 00:00:00 2001
From: Cory Snider <csnider@mirantis.com>
Date: Wed, 4 Sep 2024 16:18:54 -0400
Subject: [PATCH] Linting: disable troublesome lint rules

Unused named function parameters and shadowed variables are trivial code
quality issues which are not worth addressing on a long-term maintenance
branch without much activity aside from backports.

Signed-off-by: Cory Snider <csnider@mirantis.com>
---
 .golangci.yml | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/.golangci.yml b/.golangci.yml
index 2a0499d968..32ef636f69 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -36,17 +36,16 @@ linters-settings:
             desc: The io/ioutil package has been deprecated, see https://go.dev/doc/go1.16#ioutil
   gocyclo:
     min-complexity: 16
-  govet:
-    enable:
-      - shadow
-    settings:
-      shadow:
-        strict: true
   lll:
     line-length: 200
   nakedret:
     command: nakedret
     pattern: ^(?P<path>.*?\\.go):(?P<line>\\d+)\\s*(?P<message>.*)$
+  revive:
+    rules:
+      - name: unused-parameter
+        severity: warning
+        disabled: true
 
 issues:
   # The default exclusion rules are a bit too permissive, so copying the relevant ones below