Merge pull request #23 from WillAbides/g306

more lint fixes
WillAbides · May 16, 2023 · a1ff95b · a1ff95b
2 parents 760ce02 + d1874b4
commit a1ff95b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/.golangci.yml b/.golangci.yml
@@ -12,6 +12,7 @@ issues:
     - "G302: Expect file permissions to be 0600 or less"
     - "G304: Potential file inclusion via variable"
     - "G204: Subprocess launched with a potential tainted input or cmd arguments"
+    - "G306: Expect WriteFile permissions to be 0600 or less"
 linters:
   enable:
     - gosec

diff --git a/cmd/benchdiff/internal/benchdiff.go b/cmd/benchdiff/internal/benchdiff.go
@@ -101,7 +101,7 @@ stderr: %s`, cmd.String(), exitErr.ExitCode(), bufStderr.String())
 }
 
 func (c *Benchdiff) runBenchmark(ref, filename, extraArgs string, pause time.Duration, force bool) error {
-	cmd := exec.Command(c.BenchCmd, strings.Fields(c.BenchArgs+" "+extraArgs)...) //nolint:gosec // this is fine
+	cmd := exec.Command(c.BenchCmd, strings.Fields(c.BenchArgs+" "+extraArgs)...)
 
 	stdlib := false
 	if rootPath, err := runGitCmd(c.debug(), c.gitCmd(), c.Path, "rev-parse", "--show-toplevel"); err == nil {