return errors directly, less lines of code

Signed-off-by: Josh Dolitsky <josh@dolit.ski>

internal/experimental/action/push.go

@@ -67,6 +67,5 @@ func (p *Push) Run(chartRef string, remote string) (string, error) {
 		c.Options = append(c.Options, pusher.WithRegistryClient(p.cfg.RegistryClient))
 	}
 
-	err := c.UploadTo(chartRef, remote)
+	return out.String(), c.UploadTo(chartRef, remote)
-	return out.String(), err
 }

internal/experimental/pusher/ocipusher.go

@@ -67,11 +67,11 @@ func (pusher *OCIPusher) push(chartRef, href string) error {
 
 	var pushOpts []registry.PushOption
 	provRef := fmt.Sprintf("%s.prov", chartRef)
-	if _, err := os.Stat(provRef); err == nil {
+	if provBytes, err := ioutil.ReadFile(provRef); err != nil {
-		provBytes, err := ioutil.ReadFile(provRef)
+		if err != os.ErrNotExist { // ignore error if .prov does not exist
-		if err != nil {
 			return err
 		}
+	} else {
 		pushOpts = append(pushOpts, registry.PushOptProvData(provBytes))
 	}
 
@@ -80,13 +80,9 @@ func (pusher *OCIPusher) push(chartRef, href string) error {
 		meta.Metadata.Version)
 
 	_, err = client.Push(chartBytes, ref, pushOpts...)
-	if err != nil {
 	return err
 }
 
-	return nil
-}
-
 // NewOCIPusher constructs a valid OCI client as a Pusher
 func NewOCIPusher(ops ...Option) (Pusher, error) {
 	registryClient, err := registry.NewClient()