chore(linter): fix linter errors, remove deprecated allow leading spa…

…ces (#61)
go-vela · Dec 13, 2023 · e94dd43 · e94dd43
1 parent cbbb4b9
commit e94dd43
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/.golangci.yml b/.golangci.yml
@@ -43,7 +43,6 @@ linters-settings:
 
   # https://github.com/golangci/golangci-lint/blob/master/pkg/golinters/nolintlint
   nolintlint:
-    allow-leading-space: true     # allow non-"machine-readable" format (ie. with leading space) 
     allow-unused: false           # allow nolint directives that don't address a linting issue
     require-explanation: true     # require an explanation for nolint directives
     require-specific: true        # require nolint directives to be specific about which linter is being skipped

diff --git a/pkg/binarywrapper/binarywrapper.go b/pkg/binarywrapper/binarywrapper.go
@@ -160,7 +160,7 @@ func (p *Plugin) Exec() error {
 				logrus.Error(errorBuffer.String())
 			}
 
-			return fmt.Errorf("%w: %s", ErrExec, err)
+			return fmt.Errorf("%w: %w", ErrExec, err)
 		}
 
 		if outBuffer.Len() > 0 {
@@ -181,7 +181,7 @@ func (p *Plugin) Exec() error {
 			if errors.Is(err, os.ErrNotExist) {
 				return fmt.Errorf("%w: %s", ErrMissingBinary, p.Binary())
 			}
-			return fmt.Errorf("%w: %s", ErrExec, err)
+			return fmt.Errorf("%w: %w", ErrExec, err)
 		}
 	} else {
 		return fmt.Errorf("%w: %d", ErrUnknownExecStyle, p.ExecStyle)