Skip to content

Commit

Permalink
Fix flag shorthand conflict. (#134)
Browse files Browse the repository at this point in the history
Both the "verbose" flag in the cli root and the "version" flag in the
read command use the "v" shorthand, which causes a panic on read. This
patch drops the shorthand for the "verbose" flag.
  • Loading branch information
jmcarp authored and nickatsegment committed Sep 13, 2018
1 parent f08dc82 commit e61fff1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion cmd/root.go
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ var RootCmd = &cobra.Command{

func init() {
RootCmd.PersistentFlags().IntVarP(&numRetries, "retries", "r", DefaultNumRetries, "For SSM, the number of retries we'll make before giving up")
RootCmd.PersistentFlags().BoolVarP(&verbose, "verbose", "v", false, "Print more information to STDOUT")
RootCmd.PersistentFlags().BoolVarP(&verbose, "verbose", "", false, "Print more information to STDOUT")
}

// Execute adds all child commands to the root command sets flags appropriately.
Expand Down

0 comments on commit e61fff1

Please sign in to comment.