refactor: pipe.Skipf

internal/http/http.go

@@ -145,7 +145,7 @@ func getPassword(ctx *context.Context, upload *config.Upload, kind string) strin
 }
 
 func misconfigured(kind string, upload *config.Upload, reason string) error {
-	return pipe.Skip(fmt.Sprintf("%s section '%s' is not configured properly (%s)", kind, upload.Name, reason))
+	return pipe.Skipf("%s section '%s' is not configured properly (%s)", kind, upload.Name, reason)
 }
 
 // ResponseChecker is a function capable of validating an http server response.

internal/pipe/pipe.go

@@ -3,6 +3,7 @@ package pipe
 
 import (
 	"errors"
+	"fmt"
 	"strings"
 )
 
@@ -45,6 +46,11 @@ func Skip(reason string) ErrSkip {
 	return ErrSkip{reason: reason}
 }
 
+// Skipf skips this pipe with the given reason.
+func Skipf(format string, a ...any) ErrSkip {
+	return Skip(fmt.Sprintf(format, a...))
+}
+
 // SkipMemento remembers previous skip errors so you can return them all at once later.
 type SkipMemento struct {
 	skips []string

internal/pipe/universalbinary/universalbinary.go

@@ -148,7 +148,7 @@ func makeUniversalBinary(ctx *context.Context, opts *build.Options, unibin confi
 
 	binaries := ctx.Artifacts.Filter(filterFor(unibin)).List()
 	if len(binaries) == 0 {
-		return pipe.Skip(fmt.Sprintf("no darwin binaries found with id %q", unibin.ID))
+		return pipe.Skipf("no darwin binaries found with id %q", unibin.ID)
 	}
 
 	log.WithField("id", unibin.ID).