Feedback fix

cmd/create/create.go

@@ -25,14 +25,14 @@ func Register(ctx *context.StoolContext) {
 		Description: "Create update for application",
 		Flags: []cli.Flag{
 			cli.StringFlag{
-				Name:        "sourceDir, sd",
-				Usage:       "Path to game(short form sd)",
+				Name:        "source, s",
+				Usage:       "Path to game",
 				Value:       "",
 				Destination: &args.SourceDir,
 			},
 			cli.StringFlag{
-				Name:        "outputDir, od",
-				Usage:       "Path to game(short form sd)",
+				Name:        "output, o",
+				Usage:       "Path to game",
 				Value:       "",
 				Destination: &args.TargetDir,
 			},

cmd/diff/diff.go

@@ -23,20 +23,20 @@ func Register(ctx *context.StoolContext) {
 
 		Flags: []cli.Flag{
 			cli.StringFlag{
-				Name:        "sourceOldDir, sod",
-				Usage:       "Path to old files (short form sod)",
+				Name:        "old, o",
+				Usage:       "Path to old files",
 				Value:       "",
 				Destination: &args.SourceOldDir,
 			},
 			cli.StringFlag{
-				Name:        "sourceNewDir, snd",
-				Usage:       "Path to new files (short form snd)",
+				Name:        "new, n",
+				Usage:       "Path to new files",
 				Value:       "",
 				Destination: &args.SourceNewDir,
 			},
 			cli.StringFlag{
-				Name:        "outputDir, od",
-				Usage:       "Path to diff files (short form od)",
+				Name:        "patch, p",
+				Usage:       "Path to patch files",
 				Value:       "",
 				Destination: &args.OutputDiffDir,
 			},

cmd/push/push.go

@@ -22,8 +22,8 @@ func Register(ctx *context.StoolContext) {
 
 		Flags: []cli.Flag{
 			cli.StringFlag{
-				Name:        "sourceDir, sd",
-				Usage:       "Path to game(short form sd)",
+				Name:        "source, s",
+				Usage:       "Path to game",
 				Value:       "",
 				Destination: &args.SourceDir,
 			},

cmd/torrent/torrent.go

@@ -33,14 +33,13 @@ func Register(ctx *context.StoolContext) {
 
 		Flags: []cli.Flag{
 			cli.StringFlag{
-				Name:        "sourceDir, sd",
-				Usage:       "Path to game(short form sd)",
+				Name:        "source, s",
+				Usage:       "Path to game",
 				Value:       "",
 				Destination: &args.SourceDir,
 			},
-
 			cli.StringFlag{
-				Name:        "targetFile, tf",
+				Name:        "target, t",
 				Usage:       "Path for new torrent file",
 				Value:       "",
 				Destination: &args.TargetFile,
@@ -51,12 +50,12 @@ func Register(ctx *context.StoolContext) {
 				Usage: "Slice of torrent web seeds",
 			},
 			cli.StringSliceFlag{
-				Name:  "announce-list, a",
+				Name:  "announce-list, al",
 				Value: &args.AnnounceList,
 				Usage: "Slice of announce server url",
 			},
 			cli.Int64Flag{
-				Name:        "pieceLength, pl",
+				Name:        "piece-length, pl",
 				Usage:       "Torrent piece length",
 				Value:       512,
 				Destination: &args.PieceLength,

updater/bindiff.go

@@ -1,7 +1,6 @@
 package updater
 
 import (
-	"fmt"
 	"os"
 	"path/filepath"
 
@@ -29,8 +28,7 @@ func CreateBinDiff(SourceOldDir string, SourceNewDir string, OutputDiffDir strin
 	for pathNewFile := range f {
 		relativePath, err := filepath.Rel(SourceNewDir, pathNewFile)
 		if err != nil {
-			fmt.Println(err.Error())
-			//return err
+			return err
 		}
 
 		pathOldFile := filepath.Join(SourceOldDir, relativePath)
@@ -46,8 +44,7 @@ func CreateBinDiff(SourceOldDir string, SourceNewDir string, OutputDiffDir strin
 			cmp := equalfile.New(nil, equalfile.Options{})
 			equal, err := cmp.CompareFile(pathOldFile, pathNewFile)
 			if err != nil {
-				fmt.Println(err.Error())
-				//return err
+				return err
 			}
 
 			if equal {
@@ -59,14 +56,12 @@ func CreateBinDiff(SourceOldDir string, SourceNewDir string, OutputDiffDir strin
 		if _, err := os.Stat(pathDiff); os.IsNotExist(err) {
 			err = os.MkdirAll(pathDiff, 0777)
 			if err != nil {
-				fmt.Println(err.Error())
-				//return err
+				return err
 			}
 		}
 		err = xdelta.EncodeDiff(pathOldFile, pathNewFile, pathDiffFile)
 		if err != nil {
-			fmt.Println(err.Error())
-			//return err
+			return err
 		}
 	}