[chore]: linting (#2840)

Signed-off-by: Fernandez Ludovic <[email protected]>

Author: ldez

.golangci.yml

@@ -1,8 +1,5 @@
 run:
-  skip-dirs:
-    - helpers/
-  go: 1.22
-  deadline: 5m
+  timeout: 5m
 
 linters-settings:
   gocyclo:
@@ -80,6 +77,14 @@ linters:
   - wsl
 
 issues:
-  exclude-use-default: false # disable filtering of defaults for better zero-issue policy
-  max-per-linter: 0 # disable limit; report all issues of a linter
+  max-issues-per-linter: 0 # disable limit; report all issues of a linter
   max-same-issues: 0 # disable limit; report all issues of the same issue
+  exclude-use-default: false # disable filtering of defaults for better zero-issue policy
+  exclude-dirs:
+    - helpers/
+
+output:
+  sort-results: true
+  sort-order:
+    - linter
+    - file

internal/action/clihelper_test.go

@@ -65,8 +65,6 @@ func TestParseArgs(t *testing.T) {
 			kvOut:  map[string]string{"baz": "bam"},
 		},
 	} {
-		tc := tc
-
 		t.Run(tc.name, func(t *testing.T) {
 			if tc.argOut == nil {
 				tc.argOut = argList{}

internal/action/commands_test.go

@@ -48,7 +48,6 @@ func TestCommands(t *testing.T) {
 	ctx = act.cfg.WithConfig(ctx) //nolint:ineffassign
 
 	for _, cmd := range act.GetCommands() {
-		cmd := cmd
 		t.Run(cmd.Name, func(t *testing.T) {
 			testCommand(t, cmd)
 		})

internal/action/generate_test.go

@@ -244,8 +244,6 @@ func TestKeyAndLength(t *testing.T) {
 			length: "",
 		},
 	} {
-		tc := tc
-
 		t.Run(fmt.Sprintf("%v", tc.in), func(t *testing.T) {
 			app := cli.NewApp()
 			fs := flag.NewFlagSet("default", flag.ContinueOnError)
@@ -272,7 +270,6 @@ func TestExtractEmails(t *testing.T) {
 			out: []string{"[email protected]", "[email protected]"},
 		},
 	} {
-		tc := tc
 		t.Run(fmt.Sprintf("%v", tc.in), func(t *testing.T) {
 			assert.Equal(t, tc.out, extractEmails(tc.in))
 		})
@@ -292,7 +289,6 @@ func TestExtractDomains(t *testing.T) {
 			out: []string{"gmail.com", "live.com", "web.de"},
 		},
 	} {
-		tc := tc
 		t.Run(fmt.Sprintf("%v", tc.in), func(t *testing.T) {
 			assert.Equal(t, tc.out, extractDomains(tc.in))
 		})
@@ -312,7 +308,6 @@ func TestUniq(t *testing.T) {
 			out: []string{"bar", "foo"},
 		},
 	} {
-		tc := tc
 		t.Run(fmt.Sprintf("%v", tc.in), func(t *testing.T) {
 			assert.Equal(t, tc.out, uniq(tc.in))
 		})
@@ -339,7 +334,6 @@ func TestFilterPrefix(t *testing.T) {
 			out:    []string{"foo/bar", "foo/baz"},
 		},
 	} {
-		tc := tc
 		t.Run(fmt.Sprintf("%v", tc.in), func(t *testing.T) {
 			assert.Equal(t, tc.out, filterPrefix(tc.in, tc.prefix))
 		})
@@ -372,7 +366,6 @@ func TestDefaultLengthFromEnv(t *testing.T) {
 			{in: "abc", expected: config.DefaultPasswordLength, custom: false},
 			{in: "-1", expected: config.DefaultPasswordLength, custom: false},
 		} {
-			tc := tc
 			t.Setenv(pwLengthEnvName, tc.in)
 			actual, isCustom := config.DefaultPasswordLengthFromEnv(ctx)
 			assert.Equal(t, tc.expected, actual)

internal/action/init_test.go

@@ -98,8 +98,6 @@ func TestInitParseContext(t *testing.T) {
 			},
 		},
 	} {
-		tc := tc
-
 		t.Run(tc.name, func(t *testing.T) {
 			c := gptest.CliCtxWithFlags(config.NewContextInMemory(), t, tc.flags)
 			require.NoError(t, tc.check(initParseContext(c.Context, c)), tc.name)

internal/action/pwgen/pwgen.go

@@ -71,7 +71,7 @@ func xkcdGen(c *cli.Context, length, num int) error {
 		numbers = c.Bool("xkcdnumbers")
 	}
 
-	for i := 0; i < num; i++ {
+	for range num {
 		s, err := xkcdgen.RandomLengthDelim(length, sep, lang, capitalize, numbers)
 		if err != nil {
 			return err
@@ -109,8 +109,8 @@ func pwGen(c *cli.Context, pwLen, pwNum int) error {
 		charset += pwgen.Syms
 	}
 
-	for i := 0; i < pwNum; i++ {
-		for j := 0; j < perLine; j++ {
+	for range pwNum {
+		for range perLine {
 			ctx := out.WithNewline(ctx, false)
 			out.Print(ctx, pwgen.GeneratePasswordCharset(pwLen, charset))
 			out.Print(ctx, " ")

internal/action/recipients.go

@@ -141,14 +141,13 @@ func (s *Action) RecipientsAdd(c *cli.Context) error {
 			continue
 		}
 
-		recp := r
 		debug.Log("found recipients for %q: %+v", r, keys)
 
-		if !termio.AskForConfirmation(ctx, fmt.Sprintf("Do you want to add %q (key %q) as a recipient to the store %q?", crypto.FormatKey(ctx, recp, ""), recp, store)) {
+		if !termio.AskForConfirmation(ctx, fmt.Sprintf("Do you want to add %q (key %q) as a recipient to the store %q?", crypto.FormatKey(ctx, r, ""), r, store)) {
 			continue
 		}
 
-		if err := s.Store.AddRecipient(ctx, store, recp); err != nil {
+		if err := s.Store.AddRecipient(ctx, store, r); err != nil {
 			return exit.Error(exit.Recipients, err, "failed to add recipient %q: %s", r, err)
 		}
 		added++
@@ -201,7 +200,7 @@ func (s *Action) RecipientsRemove(c *cli.Context) error {
 			}
 		}
 
-		// if we a literal recipient (e.g. ID) is given just remove that w/o any kind of lookups.
+		// if a literal recipient (e.g. ID) is given just remove that w/o any kind of lookups.
 		if set.Contains(knownRecipients, r) {
 			debug.Log("Removing %q from %q (direct)", r, store)
 			if err := s.Store.RemoveRecipient(ctx, store, r); err != nil {
@@ -213,7 +212,7 @@ func (s *Action) RecipientsRemove(c *cli.Context) error {
 			continue
 		}
 
-		// look up the full key ID of the given recipient (could be email or any kinf of short ID)
+		// look up the full key ID of the given recipient (could be email or any kind of short ID)
 		keys, err := crypto.FindRecipients(ctx, r)
 		if err != nil {
 			out.Printf(ctx, "WARNING: Failed to list public key %q: %s", r, err)
@@ -235,7 +234,7 @@ func (s *Action) RecipientsRemove(c *cli.Context) error {
 			continue
 		}
 
-		recp := r
+		recp := r //nolint:copyloopvar
 		if len(keys) > 0 {
 			if nr := crypto.Fingerprint(ctx, keys[0]); nr != "" {
 				debug.Log("Fingerprint translated %q into %q", keys[0], nr)

internal/audit/audit.go

@@ -149,7 +149,7 @@ func (a *Auditor) Batch(ctx context.Context, secrets []string) (*Report, error)
 	debug.Log("launching %d audit workers", maxJobs)
 
 	done := make(chan struct{}, maxJobs)
-	for jobs := 0; jobs < maxJobs; jobs++ {
+	for range maxJobs {
 		go a.audit(ctx, pending, done)
 	}
 
@@ -166,7 +166,7 @@ func (a *Auditor) Batch(ctx context.Context, secrets []string) (*Report, error)
 		bar.Inc()
 	}
 
-	for i := 0; i < maxJobs; i++ {
+	for range maxJobs {
 		<-done
 	}
 	bar.Done()
@@ -228,8 +228,6 @@ func (a *Auditor) auditSecret(ctx context.Context, secret string) {
 	// pass the secret to all validators.
 	var wg sync.WaitGroup
 	for _, v := range a.v {
-		v := v
-
 		wg.Add(1)
 		go func() {
 			defer wg.Done()

internal/backend/crypto/gpg/colons/parse_colons_test.go

@@ -41,7 +41,6 @@ func TestParseColonIdentity(t *testing.T) {
 			email:   "",
 		},
 	} {
-		tc := tc
 		t.Run(tc.in, func(t *testing.T) {
 			t.Parallel()
 

internal/backend/crypto/gpg/gpgconf/version_test.go

@@ -39,8 +39,6 @@ func TestSort(t *testing.T) {
 			},
 		},
 	} {
-		tc := tc
-
 		t.Run(tc.name, func(t *testing.T) {
 			t.Parallel()