[chore] use the same version of golangci-lint

.github/workflows/golangci-lint.yml

@@ -43,5 +43,9 @@ jobs:
       - name: golangci-lint
         uses: golangci/golangci-lint-action@aaa42aa0628b4ae2578232a66b541047968fac86 # v6.1.0
         with:
-          version: v1.59.1 # we have a list of linters in our .golangci.yml config file
+          # Note: there are 2 different version of golangci-lint used inside the project.
+          # https://github.com/gopasspw/gopass/blob/master/.github/workflows/build.yml#L65
+          # https://github.com/gopasspw/gopass/blob/master/.github/workflows/golangci-lint.yml#L46
+          # https://github.com/gopasspw/gopass/blob/master/Makefile#L136
+          version: v1.61.0 # we have a list of linters in our .golangci.yml config file
           only-new-issues: true

.golangci.yml

@@ -36,7 +36,7 @@ linters:
   - errname
   - errorlint
   - exhaustive
-  - exportloopref
+  # - exportloopref # deprecated since Go 1.22
   - forcetypeassert
   - funlen
   - ginkgolinter
@@ -90,6 +90,7 @@ issues:
     - helpers/
 
 output:
+  show-stats: true
   sort-results: true
   sort-order:
     - linter

Makefile

@@ -131,6 +131,10 @@ crosscompile:
 codequality:
 	@echo ">> CODE QUALITY"
 
+	# Note: there are 2 different version of golangci-lint used inside the project.
+	# https://github.com/gopasspw/gopass/blob/master/.github/workflows/build.yml#L65
+	# https://github.com/gopasspw/gopass/blob/master/.github/workflows/golangci-lint.yml#L46
+	# https://github.com/gopasspw/gopass/blob/master/Makefile#L136
 	@echo -n "     GOLANGCI-LINT "
 	@which golangci-lint > /dev/null; if [ $$? -ne 0 ]; then \
 		$(GO) install github.com/golangci/golangci-lint/cmd/golangci-lint@latest; \

internal/action/show.go

@@ -197,7 +197,7 @@ func (s *Action) showHandleOutput(ctx context.Context, name string, sec gopass.S
 			out.Warning(ctx, "show.safecontent=true. Use -f to display password, if any")
 		}
 
-		return exit.Error(exit.NotFound, store.ErrEmptySecret, store.ErrEmptySecret.Error())
+		return exit.Error(exit.NotFound, store.ErrEmptySecret, "%v", store.ErrEmptySecret)
 	}
 
 	if IsPrintQR(ctx) && IsQRBody(ctx) {
@@ -244,7 +244,7 @@ func (s *Action) showGetContent(ctx context.Context, sec gopass.Secret) (string,
 		key := GetKey(ctx)
 		values, found := sec.Values(key)
 		if !found {
-			return "", "", exit.Error(exit.NotFound, store.ErrNoKey, store.ErrNoKey.Error())
+			return "", "", exit.Error(exit.NotFound, store.ErrNoKey, "%v", store.ErrNoKey)
 		}
 		val := strings.Join(values, "\n")
 
@@ -264,7 +264,7 @@ func (s *Action) showGetContent(ctx context.Context, sec gopass.Secret) (string,
 	}
 	if IsPasswordOnly(ctx) {
 		if pw == "" && fullBody != "" {
-			return "", "", exit.Error(exit.NotFound, store.ErrNoPassword, store.ErrNoPassword.Error())
+			return "", "", exit.Error(exit.NotFound, store.ErrNoPassword, "%v", store.ErrNoPassword)
 		}
 
 		return pw, pw, nil

internal/backend/crypto/gpg/key.go

@@ -74,7 +74,7 @@ func (k Key) String() string {
 
 	out += "\n      Key fingerprint = " + k.Fingerprint
 	for _, id := range k.Identities {
-		out += fmt.Sprintf("\n" + id.String())
+		out += fmt.Sprintf("\n%s", id)
 	}
 
 	return out

internal/store/leaf/recipients_test.go

@@ -137,7 +137,7 @@ func TestSaveRecipients(t *testing.T) {
 	sort.Strings(foundRecs)
 
 	ids := rs.IDs()
-	for i := range len(ids) {
+	for i := range ids {
 		if i >= len(foundRecs) {
 			t.Errorf("Read too few recipients")
 

pkg/clipboard/clipboard.go

@@ -18,7 +18,8 @@ var (
 	// -ldflags=='-X github.com/gopasspw/gopass/pkg/clipboard.Helpers=termux-api'.
 	Helpers = "xsel or xclip"
 	// ErrNotSupported is returned when the clipboard is not accessible.
-	ErrNotSupported = fmt.Errorf("WARNING: No clipboard available. Install " + Helpers + ", provide $GOPASS_CLIPBOARD_COPY_CMD and $GOPASS_CLIPBOARD_CLEAR_CMD or use -f to print to console")
+	ErrNotSupported = fmt.Errorf("WARNING: No clipboard available. "+
+		"Install %s, provide $GOPASS_CLIPBOARD_COPY_CMD and $GOPASS_CLIPBOARD_CLEAR_CMD or use -f to print to console", Helpers)
 )
 
 // CopyTo copies the given data to the clipboard and enqueues automatic

tests/list_test.go

@@ -55,8 +55,6 @@ foo/
 
 // regression test for #1628.
 func TestListRegressions1628(t *testing.T) {
-	t.Parallel()
-
 	ts := newTester(t)
 	defer ts.teardown()
 

tests/tester.go

@@ -38,6 +38,7 @@ type tester struct {
 	resetFn   func()
 }
 
+// newTester is not compatible with t.Parallel because it uses t.Setenv.
 func newTester(t *testing.T) *tester {
 	t.Helper()
 
@@ -77,13 +78,13 @@ func newTester(t *testing.T) *tester {
 
 	// prepare ENVIRONMENT
 	ts.resetFn = gptest.UnsetVars("GNUPGHOME", "GOPASS_DEBUG", "NO_COLOR", "GOPASS_CONFIG", "GOPASS_NO_NOTIFY", "GOPASS_HOMEDIR")
-	require.NoError(t, os.Setenv("GNUPGHOME", ts.gpgDir()))
-	require.NoError(t, os.Setenv("GOPASS_DEBUG", ""))
-	require.NoError(t, os.Setenv("NO_COLOR", "true"))
-	require.NoError(t, os.Setenv("GOPASS_CONFIG_NOSYSTEM", "true"))
-	require.NoError(t, os.Setenv("GOPASS_CONFIG_NO_MIGRATE", "true"))
-	require.NoError(t, os.Setenv("GOPASS_NO_NOTIFY", "true"))
-	require.NoError(t, os.Setenv("GOPASS_HOMEDIR", td))
+	t.Setenv("GNUPGHOME", ts.gpgDir())
+	t.Setenv("GOPASS_DEBUG", "")
+	t.Setenv("NO_COLOR", "true")
+	t.Setenv("GOPASS_CONFIG_NOSYSTEM", "true")
+	t.Setenv("GOPASS_CONFIG_NO_MIGRATE", "true")
+	t.Setenv("GOPASS_NO_NOTIFY", "true")
+	t.Setenv("GOPASS_HOMEDIR", td)
 
 	// write config
 	require.NoError(t, os.MkdirAll(filepath.Dir(ts.gopassConfig()), 0o700))