chore: Bump golangci-lint version

twpayne · Nov 11, 2024 · 16f4753 · 16f4753
1 parent aca64e8
commit 16f4753
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 6 deletions.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -21,7 +21,7 @@ env:
   FIND_TYPOS_VERSION: 0.0.3 # https://github.com/twpayne/find-typos/tags
   GO_VERSION: 1.23.3 # https://go.dev/doc/devel/release
   GOFUMPT_VERSION: 0.7.0 # https://github.com/mvdan/gofumpt/releases
-  GOLANGCI_LINT_VERSION: 1.61.0 # https://github.com/golangci/golangci-lint/releases
+  GOLANGCI_LINT_VERSION: 1.62.0 # https://github.com/golangci/golangci-lint/releases
   GOLINES_VERSION: 0.12.2 # https://github.com/segmentio/golines/releases
   GORELEASER_VERSION: 2.4.4 # https://github.com/goreleaser/goreleaser/releases
   GOVERSIONINFO_VERSION: 1.4.1 # https://github.com/josephspurrier/goversioninfo/releases

diff --git a/.golangci.yml b/.golangci.yml
@@ -35,6 +35,7 @@ linters:
   - gosmopolitan
   - govet
   - grouper
+  - iface
   - importas
   - inamedparam
   - ineffassign
@@ -92,7 +93,6 @@ linters:
   - gocyclo
   - godox
   - goheader
-  - gomnd
   - gomoddirectives
   - gosec
   - ireturn
@@ -105,6 +105,7 @@ linters:
   - nlreturn
   - nonamedreturns
   - paralleltest
+  - recvcheck
   - testpackage
   - tparallel
   - varnamelen

diff --git a/internal/chezmoilog/chezmoilog.go b/internal/chezmoilog/chezmoilog.go
@@ -21,9 +21,9 @@ type OSExecCmdLogValuer struct {
 	*exec.Cmd
 }
 
-// An OSExecExitErrorLogValuer wraps an *os/exec.ExitError and adds
+// An OSExecExitLogValuerError wraps an *os/exec.ExitError and adds
 // log/slog.LogValuer.
-type OSExecExitErrorLogValuer struct {
+type OSExecExitLogValuerError struct {
 	*exec.ExitError
 }
 
@@ -52,7 +52,7 @@ func (cmd OSExecCmdLogValuer) LogValuer() slog.Value {
 }
 
 // LogValuer implements log/slog.LogValuer.LogValue.
-func (err OSExecExitErrorLogValuer) LogValuer() slog.Value {
+func (err OSExecExitLogValuerError) LogValuer() slog.Value {
 	attrs := []slog.Attr{
 		slog.Any("processState", OSProcessStateLogValuer{err.ExitError.ProcessState}),
 	}

diff --git a/internal/cmd/main_test.go b/internal/cmd/main_test.go
@@ -588,7 +588,7 @@ func cmdMockCommand(ts *testscript.TestScript, neg bool, args []string) {
 		},
 		"quote":          strconv.Quote,
 		"renderResponse": renderResponseFunc,
-		"replaceAll": func(old, new, s string) string { //nolint:predeclared
+		"replaceAll": func(old, new, s string) string { //nolint:predeclared,revive
 			return strings.ReplaceAll(s, old, new)
 		},
 	}).Parse(templateText)