diff --git a/cli/catalogue/catalogue.go b/cli/catalogue/catalogue.go index 50cb8d2d..1deee22b 100644 --- a/cli/catalogue/catalogue.go +++ b/cli/catalogue/catalogue.go @@ -63,6 +63,7 @@ var catalogueGenerateCommand = &cli.Command{ internal.PushFlag, internal.CommitFlag, internal.CommitMessageFlag, + internal.DryFlag, }, Description: ` This command generates a new copy of the recipe catalogue which can be found on: @@ -233,7 +234,7 @@ A new catalogue copy can be published to the recipes repository by passing the } } - if err := gitPkg.Commit("**.json", internal.CommitMessage, false); err != nil { + if err := gitPkg.Commit("**.json", internal.CommitMessage, internal.Dry, internal.Push); err != nil { logrus.Fatal(err) } } diff --git a/cli/recipe/release.go b/cli/recipe/release.go index 18a61965..693185eb 100644 --- a/cli/recipe/release.go +++ b/cli/recipe/release.go @@ -155,7 +155,7 @@ You may invoke this command in "wizard" mode and be prompted for input: logrus.Fatal("no commit message specified?") } - if err := gitPkg.Commit("compose.**yml", internal.CommitMessage, internal.Dry); err != nil { + if err := gitPkg.Commit("compose.**yml", internal.CommitMessage, internal.Dry, false); err != nil { logrus.Fatal(err) } diff --git a/pkg/git/commit.go b/pkg/git/commit.go index 68e114db..981efe8d 100644 --- a/pkg/git/commit.go +++ b/pkg/git/commit.go @@ -10,7 +10,7 @@ import ( ) // Commit runs a git commit -func Commit(glob, commitMessage string, dryRun bool) error { +func Commit(glob, commitMessage string, dryRun, push bool) error { if commitMessage == "" { return fmt.Errorf("no commit message specified?") } @@ -55,7 +55,7 @@ func Commit(glob, commitMessage string, dryRun bool) error { logrus.Info("dry run: no changes commited") } - if !dryRun { + if !dryRun && push { if err := commitRepo.Push(&git.PushOptions{}); err != nil { return err }