chore: update golangci-lint

twpayne · Apr 17, 2024 · 930fbdb · 930fbdb
1 parent cf96da2
commit 930fbdb
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 4 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -33,4 +33,4 @@ jobs:
     - uses: actions/setup-go@0c52d547c9bc32b1aa3301fd7a9cb496313a4491
     - uses: golangci/golangci-lint-action@3cfe3a4abbb849e10058ce4af15d205b6da42804
       with:
-        version: v1.54.2
+        version: v1.57.2
diff --git a/.golangci.yml b/.golangci.yml
@@ -39,6 +39,7 @@ linters:
   - govet
   - grouper
   - importas
+  - inamedparam
   - ineffassign
   - interfacebloat
   - ireturn
@@ -55,17 +56,23 @@ linters:
   - nonamedreturns
   - nosprintfhostport
   - paralleltest
+  - perfsprint
   - prealloc
   - predeclared
   - promlinter
+  - protogetter
   - reassign
+  - revive
   - rowserrcheck
+  - sloglint
+  - spancheck
   - sqlclosecheck
   - staticcheck
   - tagalign
   - tagliatelle
   - tenv
   - testableexamples
+  - testifylint
   - thelper
   - tparallel
   - typecheck
@@ -93,7 +100,6 @@ linters:
   - maligned
   - nestif
   - nlreturn
-  - revive
   - stylecheck
   - testpackage
   - varnamelen

diff --git a/generator.go b/generator.go
@@ -19,7 +19,7 @@ import (
 var (
 	SkipDir = fs.SkipDir
 	//lint:ignore ST1012 SkipFile is not an error
-	SkipFile = errors.New("skip file") //nolint:errname
+	SkipFile = errors.New("skip file") //nolint:errname,revive
 )
 
 // A Generator observes XML documents and generates Go structs into which the

diff --git a/internal/tests/gml/gml_test.go b/internal/tests/gml/gml_test.go
@@ -42,7 +42,7 @@ func TestGML(t *testing.T) {
 	zipReader, err := zip.NewReader(file, fileInfo.Size())
 	assert.NoError(t, err)
 
-	assert.NoError(t, generator.ObserveFS(zipReader, "ets-gml32-master/src/test/resources", func(path string, dirEntry fs.DirEntry, err error) error {
+	assert.NoError(t, generator.ObserveFS(zipReader, "ets-gml32-master/src/test/resources", func(path string, _ fs.DirEntry, err error) error {
 		switch {
 		case err != nil:
 			return err