Skip to content

Commit

Permalink
bugfix: Missing confirmation input message (#224)
Browse files Browse the repository at this point in the history
* Remove duplicated calls to RawOutput

* Upgrade tsuru cmd dependency
  • Loading branch information
gvicentin authored May 2, 2024
1 parent ccbf4c7 commit 59bdc44
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ require (
github.com/tsuru/gnuflag v0.0.0-20151217162021-86b8c1b864aa
github.com/tsuru/go-tsuruclient v0.0.0-20240409125509-22a1e08326f4
github.com/tsuru/tablecli v0.0.0-20190131152944-7ded8a3383c6
github.com/tsuru/tsuru v0.0.0-20240411165438-543d3861334a
github.com/tsuru/tsuru v0.0.0-20240430173126-10560a3c53fd
golang.org/x/net v0.20.0
golang.org/x/oauth2 v0.16.0
golang.org/x/sys v0.16.0
Expand Down
4 changes: 4 additions & 0 deletions go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -681,6 +681,10 @@ github.com/tsuru/tablecli v0.0.0-20190131152944-7ded8a3383c6 h1:1XDdWFAjIbCSG1Oj
github.com/tsuru/tablecli v0.0.0-20190131152944-7ded8a3383c6/go.mod h1:ztYpOhW+u1k21FEqp7nZNgpWbr0dUKok5lgGCZi+1AQ=
github.com/tsuru/tsuru v0.0.0-20240411165438-543d3861334a h1:O0fPdqtXD4RQXaptD1AjezS/4Ryo5Fxjrm38jKZRYkE=
github.com/tsuru/tsuru v0.0.0-20240411165438-543d3861334a/go.mod h1:QxMVXEjpb6mPnlaLMQdxJ3XCg+vOa4/MwZWLk3/naec=
github.com/tsuru/tsuru v0.0.0-20240426210150-8ecd7f486345 h1:A6SbQLCBGgdJOKoXUrUmiCDVLNyy+cHV3omW9vgua8M=
github.com/tsuru/tsuru v0.0.0-20240426210150-8ecd7f486345/go.mod h1:QxMVXEjpb6mPnlaLMQdxJ3XCg+vOa4/MwZWLk3/naec=
github.com/tsuru/tsuru v0.0.0-20240430173126-10560a3c53fd h1:Al7qvBMn76Pb3Ci3AzULVTNUlv2SIn6MtjrLfbM0dlg=
github.com/tsuru/tsuru v0.0.0-20240430173126-10560a3c53fd/go.mod h1:QxMVXEjpb6mPnlaLMQdxJ3XCg+vOa4/MwZWLk3/naec=
github.com/ugorji/go v1.1.4/go.mod h1:uQMGLiO92mf5W77hV/PUCpI3pbzQx3CRekS0kk+RGrc=
github.com/urfave/cli v0.0.0-20171014202726-7bc6a0acffa5/go.mod h1:70zkFmudgCuE/ngEzBv17Jvp/497gISqfk5gWijbERA=
github.com/urfave/cli v1.20.0/go.mod h1:70zkFmudgCuE/ngEzBv17Jvp/497gISqfk5gWijbERA=
Expand Down
1 change: 0 additions & 1 deletion tsuru/admin/cluster.go
Original file line number Diff line number Diff line change
Expand Up @@ -458,7 +458,6 @@ func (c *ClusterRemove) Info() *cmd.Info {
}

func (c *ClusterRemove) Run(ctx *cmd.Context) error {
ctx.RawOutput()
name := ctx.Args[0]
if !c.Confirm(ctx, fmt.Sprintf("Are you sure you want to remove cluster \"%s\"?", name)) {
return nil
Expand Down
1 change: 0 additions & 1 deletion tsuru/client/apps.go
Original file line number Diff line number Diff line change
Expand Up @@ -351,7 +351,6 @@ remove it (you are able to remove any app that you see in [[tsuru app list]]).`,
}

func (c *AppRemove) Run(context *cmd.Context) error {
context.RawOutput()
appName := c.Flags().Lookup("app").Value.String()
if appName == "" {
return errors.New("Please use the -a/--app flag to specify which app you want to remove.")
Expand Down
1 change: 0 additions & 1 deletion tsuru/client/deploy.go
Original file line number Diff line number Diff line change
Expand Up @@ -408,7 +408,6 @@ func (c *AppDeployRollback) Info() *cmd.Info {
}

func (c *AppDeployRollback) Run(context *cmd.Context) error {
context.RawOutput()
appName, err := c.AppName()
if err != nil {
return err
Expand Down

0 comments on commit 59bdc44

Please sign in to comment.