Move an error check to a more correct location.

chrisgavin · chrisgavin · commit 48a82503ba77 · 2022-05-30T11:11:46.000+01:00
diff --git a/internal/push/push.go b/internal/push/push.go
@@ -324,15 +324,15 @@ func (pushService *pushService) createOrUpdateReleaseAsset(release *github.Repos
 	}
 	log.Debugf("Uploading release asset %s...", assetPathStat.Name())
 	assetFile, err := os.Open(pushService.cacheDirectory.AssetPath(release.GetTagName(), assetPathStat.Name()))
+	if err != nil {
+		return errors.Wrap(err, "Error opening release asset.")
+	}
 	defer assetFile.Close()
 	progressReader := &ioprogress.Reader{
 		Reader:   assetFile,
 		Size:     assetPathStat.Size(),
 		DrawFunc: ioprogress.DrawTerminalf(os.Stderr, ioprogress.DrawTextFormatBytes),
 	}
-	if err != nil {
-		return errors.Wrap(err, "Error opening release asset.")
-	}
 	_, _, err = pushService.uploadReleaseAsset(release, assetPathStat, progressReader)
 	if err != nil {
 		return errors.Wrap(err, "Error uploading release asset.")
