fix linter warnings (#403)

Fixing these linter warnings:

```
WARN [config_reader] The configuration option `linters.govet.check-shadowing` is deprecated. Please enable `shadow` instead, if you are not using `enable-all`.
WARN The linter 'exportloopref' is deprecated (since v1.60.2) due to: Since Go1.22 (loopvar) this linter is no longer relevant. Replaced by copyloopvar.
```

Author: HaraldNordgren

.golangci.yml

@@ -20,7 +20,7 @@ linters:
     - depguard
     - durationcheck
     - errorlint
-    - exportloopref
+    - copyloopvar
     - gocritic
     - nakedret
     - stylecheck
@@ -37,7 +37,6 @@ linters-settings:
     errorf: false   # it's valid to use %v instead of %w
 
   govet:
-    check-shadowing: true
     enable-all: true
 
   # We have a ton of test-only packages; but make sure we keep prod deps small.

generate/imports_test.go

@@ -28,7 +28,6 @@ func TestMakeIdentifier(t *testing.T) {
 	}
 
 	for _, test := range tests {
-		test := test
 		t.Run(test.testName, func(t *testing.T) {
 			actual := makeIdentifier(test.input)
 			if actual != test.expected {

generate/names_test.go

@@ -56,7 +56,6 @@ func TestTypeNames(t *testing.T) {
 		"lower",
 	}}
 	for _, test := range tests {
-		test := test
 		t.Run(test.expectedTypeName, func(t *testing.T) {
 			prefix := newPrefixList("Operation")
 			for _, field := range test.fields {
@@ -105,7 +104,6 @@ func TestSnakeToTypeNames(t *testing.T) {
 	}}
 
 	for _, test := range tests {
-		test := test
 		t.Run(test.expectedTypeName, func(t *testing.T) {
 			prefix := newPrefixList("ServiceIPs")
 			algorithm := CasingDefault

generate/util_test.go

@@ -10,7 +10,6 @@ type test struct {
 
 func testStringFunc(t *testing.T, f func(string) string, tests []test) {
 	for _, test := range tests {
-		test := test
 		t.Run(test.name, func(t *testing.T) {
 			got := f(test.in)
 			if got != test.out {