Merge pull request #30 from writeas/T612

return errors on delete command

api/api.go

@@ -153,7 +153,7 @@ func DoUpdate(c *cli.Context, post []byte, friendlyID, token, font string, tor,
 	return nil
 }
 
-// DoDelete deletes the given post on Write.as.
+// DoDelete deletes the given post on Write.as, and removes any local references
 func DoDelete(c *cli.Context, friendlyID, token string, tor bool) error {
 	cl, _ := NewClient(c, false)
 

commands/commands.go

@@ -104,7 +104,7 @@ func CmdDelete(c *cli.Context) error {
 
 	err := api.DoDelete(c, friendlyID, token, tor)
 	if err != nil {
-		return err
+		return cli.NewExitError(fmt.Sprintf("Couldn't delete remote copy: %v", err), 1)
 	}
 
 	// Delete local file, if necessary
@@ -113,7 +113,7 @@ func CmdDelete(c *cli.Context) error {
 		// TODO: handle deleting blog posts
 		err = fileutils.DeleteFile(filepath.Join(cfg.Posts.Directory, friendlyID+api.PostFileExt))
 		if err != nil {
-			return err
+			return cli.NewExitError(fmt.Sprintf("Couldn't delete local copy: %v", err), 1)
 		}
 	}