review changes

cli/cmd/deploy/cmd.go

@@ -1,7 +1,7 @@
 package deploy
 
 import (
-	"errors"
+	"fmt"
 	"net/http"
 	"os"
 
@@ -39,7 +39,7 @@ be pushed to the organization "organization-slug-a3ecfh2b", and the app name
 			fn := cmd.HelpFunc()
 			fn(cmd, args)
 
-			return errors.New("only accepts zero or one positional arguments")
+			return fmt.Errorf("accepts only an optional [app directory] as a positional argument, you provided %d arguments", len(args))
 		}
 
 		if len(args) == 1 {

cli/cmd/deploy/deploy.go

@@ -95,6 +95,7 @@ func Deploy(ctx context.Context, apps AppService, appDir, projectDir, slug strin
 
 		return err
 	}
+	defer archive.Close()
 	task.Done()
 
 	task = output.StartTask("Uploading app archive")

cli/cmd/output/task.go

@@ -42,7 +42,7 @@ func (t *Task) Done() {
 	fmt.Println(ln + ansiGreen + "OK" + ansiReset)
 }
 
-func (t Task) Error() {
+func (t *Task) Error() {
 	ln := t.line(errorcross)
 	if !t.lineAdded {
 		fmt.Print("\r")

cli/cmd/push/push.go

@@ -168,7 +168,7 @@ func buildApp(buildID string, appPath string) (ok bool) {
 
 func uploadApp(appDir string, toolID string) (buildID string, ok bool) {
 	defer os.Remove(zipFileName)
-	task := output.StartTask("Uploading app......")
+	task := output.StartTask("Uploading app")
 
 	secrets := loadSecretsFromEnv(appDir)
 	buildID, err := pushBuild(zipFileName, string(toolID), secrets)
@@ -194,7 +194,7 @@ func uploadApp(appDir string, toolID string) (buildID string, ok bool) {
 }
 
 func prepareApp(m *manifest.Manifest) (ok bool) {
-	task := output.StartTask("Preparing upload...")
+	task := output.StartTask("Preparing upload.")
 
 	if err := archive.ZipCreate(".", zipFileName, m.Exclude); err != nil {
 		output.PrintErrorDetails("Error preparing app.", err)