refactor: don't ignore errors setting up cobra flags (#358)

scop · web-flow · commit 539589827b55 · 2025-01-30T19:50:38.000+02:00
Panic instead. These are programming errors, occurring early at startup.
diff --git a/internal/commands/account/permissions/list.go b/internal/commands/account/permissions/list.go
@@ -37,7 +37,7 @@ func (l *listCommand) InitCommand() {
 }
 
 func (l *listCommand) InitCommandWithConfig(cfg *config.Config) {
-	_ = l.Cobra().RegisterFlagCompletionFunc("username", namedargs.CompletionFunc(completion.Username{}, cfg))
+	commands.Must(l.Cobra().RegisterFlagCompletionFunc("username", namedargs.CompletionFunc(completion.Username{}, cfg)))
 }
 
 func (l *listCommand) ExecuteWithoutArguments(exec commands.Executor) (output.Output, error) {
diff --git a/internal/commands/database/index/delete.go b/internal/commands/database/index/delete.go
@@ -36,7 +36,7 @@ func (s *deleteCommand) InitCommand() {
 	flagSet.StringVar(&s.name, "name", "", "Index name")
 	s.AddFlags(flagSet)
 
-	_ = s.Cobra().MarkFlagRequired("name")
+	commands.Must(s.Cobra().MarkFlagRequired("name"))
 }
 
 // Execute implements commands.MultipleArgumentCommand
diff --git a/internal/commands/database/session/cancel.go b/internal/commands/database/session/cancel.go
@@ -41,7 +41,7 @@ func (s *cancelCommand) InitCommand() {
 	config.AddToggleFlag(flagSet, &s.terminate, "terminate", false, "Request immediate termination instead of soft cancel.")
 
 	s.AddFlags(flagSet)
-	_ = s.Cobra().MarkFlagRequired("pid")
+	commands.Must(s.Cobra().MarkFlagRequired("pid"))
 }
 
 // Execute implements commands.MultipleArgumentCommand
diff --git a/internal/commands/ipaddress/assign.go b/internal/commands/ipaddress/assign.go
@@ -57,7 +57,7 @@ func (s *assignCommand) InitCommand() {
 }
 
 func (s *assignCommand) InitCommandWithConfig(cfg *config.Config) {
-	_ = s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg))
+	commands.Must(s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg)))
 }
 
 // ExecuteWithoutArguments implements commands.NoArgumentCommand
diff --git a/internal/commands/kubernetes/create.go b/internal/commands/kubernetes/create.go
@@ -147,15 +147,15 @@ func (c *createCommand) InitCommand() {
 	config.AddToggleFlag(fs, &c.params.wait, "wait", false, "Wait for cluster to be in running state before returning.")
 	c.AddFlags(fs)
 
-	_ = c.Cobra().MarkFlagRequired("name")
-	_ = c.Cobra().MarkFlagRequired("network")
-	_ = c.Cobra().MarkFlagRequired("zone")
+	commands.Must(c.Cobra().MarkFlagRequired("name"))
+	commands.Must(c.Cobra().MarkFlagRequired("network"))
+	commands.Must(c.Cobra().MarkFlagRequired("zone"))
 }
 
 func (c *createCommand) InitCommandWithConfig(cfg *config.Config) {
-	_ = c.Cobra().RegisterFlagCompletionFunc("network", namedargs.CompletionFunc(completion.Network{}, cfg))
-	_ = c.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg))
-	_ = c.Cobra().RegisterFlagCompletionFunc("version", namedargs.CompletionFunc(completion.KubernetesVersion{}, cfg))
+	commands.Must(c.Cobra().RegisterFlagCompletionFunc("network", namedargs.CompletionFunc(completion.Network{}, cfg)))
+	commands.Must(c.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg)))
+	commands.Must(c.Cobra().RegisterFlagCompletionFunc("version", namedargs.CompletionFunc(completion.KubernetesVersion{}, cfg)))
 }
 
 // ExecuteWithoutArguments implements commands.NoArgumentCommand
diff --git a/internal/commands/kubernetes/nodegroup/create.go b/internal/commands/kubernetes/nodegroup/create.go
@@ -142,9 +142,9 @@ func (s *createCommand) InitCommand() {
 	fs := GetCreateNodeGroupFlagSet(&s.p)
 	s.AddFlags(fs)
 
-	_ = s.Cobra().MarkFlagRequired("name")
-	_ = s.Cobra().MarkFlagRequired("count")
-	_ = s.Cobra().MarkFlagRequired("plan")
+	commands.Must(s.Cobra().MarkFlagRequired("name"))
+	commands.Must(s.Cobra().MarkFlagRequired("count"))
+	commands.Must(s.Cobra().MarkFlagRequired("plan"))
 }
 
 // ExecuteSingleArgument implements commands.SingleArgumentCommand
diff --git a/internal/commands/kubernetes/nodegroup/delete.go b/internal/commands/kubernetes/nodegroup/delete.go
@@ -35,7 +35,7 @@ func (s *deleteCommand) InitCommand() {
 	flagSet.StringVar(&s.name, "name", "", "Node group name")
 	s.AddFlags(flagSet)
 
-	_ = s.Cobra().MarkFlagRequired("name")
+	commands.Must(s.Cobra().MarkFlagRequired("name"))
 }
 
 // ExecuteSingleArgument implements commands.SingleArgumentCommand
diff --git a/internal/commands/kubernetes/nodegroup/scale.go b/internal/commands/kubernetes/nodegroup/scale.go
@@ -37,8 +37,8 @@ func (s *scaleCommand) InitCommand() {
 	flagSet.IntVar(&s.count, "count", 0, "Node count")
 	s.AddFlags(flagSet)
 
-	_ = s.Cobra().MarkFlagRequired("name")
-	_ = s.Cobra().MarkFlagRequired("count")
+	commands.Must(s.Cobra().MarkFlagRequired("name"))
+	commands.Must(s.Cobra().MarkFlagRequired("count"))
 }
 
 // ExecuteSingleArgument implements commands.SingleArgumentCommand
diff --git a/internal/commands/kubernetes/nodegroup/show.go b/internal/commands/kubernetes/nodegroup/show.go
@@ -37,7 +37,7 @@ func (s *showCommand) InitCommand() {
 	flagSet.StringVar(&s.name, "name", "", "Node group name")
 	s.AddFlags(flagSet)
 
-	_ = s.Cobra().MarkFlagRequired("name")
+	commands.Must(s.Cobra().MarkFlagRequired("name"))
 }
 
 // ExecuteSingleArgument implements commands.SingleArgumentCommand
diff --git a/internal/commands/network/create.go b/internal/commands/network/create.go
@@ -55,13 +55,13 @@ func (s *createCommand) InitCommand() {
 		"  dhcp-dns: array of strings")
 
 	s.AddFlags(fs)
-	_ = s.Cobra().MarkFlagRequired("name")
-	_ = s.Cobra().MarkFlagRequired("zone")
-	_ = s.Cobra().MarkFlagRequired("ip-network")
+	commands.Must(s.Cobra().MarkFlagRequired("name"))
+	commands.Must(s.Cobra().MarkFlagRequired("zone"))
+	commands.Must(s.Cobra().MarkFlagRequired("ip-network"))
 }
 
 func (s *createCommand) InitCommandWithConfig(cfg *config.Config) {
-	_ = s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg))
+	commands.Must(s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg)))
 }
 
 // MaximumExecutions implements Command.MaximumExecutions
diff --git a/internal/commands/partner/account/create.go b/internal/commands/partner/account/create.go
@@ -45,8 +45,8 @@ func (s *createCommand) InitCommand() {
 	fs.StringVar(&s.params.ContactDetails.VATNumber, "vat-number", "", "Contact VAT number.")
 
 	s.AddFlags(fs)
-	_ = s.Cobra().MarkFlagRequired("username")
-	_ = s.Cobra().MarkFlagRequired("password")
+	commands.Must(s.Cobra().MarkFlagRequired("username"))
+	commands.Must(s.Cobra().MarkFlagRequired("password"))
 }
 
 func (s *createCommand) ExecuteWithoutArguments(exec commands.Executor) (output.Output, error) {
diff --git a/internal/commands/router/create.go b/internal/commands/router/create.go
@@ -33,7 +33,7 @@ func (s *createCommand) InitCommand() {
 	fs.StringVar(&s.name, "name", s.name, "Router name.")
 
 	s.AddFlags(fs)
-	_ = s.Cobra().MarkFlagRequired("name")
+	commands.Must(s.Cobra().MarkFlagRequired("name"))
 }
 
 // MaximumExecutions implements Command.MaximumExecutions
diff --git a/internal/commands/router/modify.go b/internal/commands/router/modify.go
@@ -36,7 +36,7 @@ func (s *modifyCommand) InitCommand() {
 	fs.StringVar(&s.name, "name", "", "New router name.")
 
 	s.AddFlags(fs)
-	_ = s.Cobra().MarkFlagRequired("name")
+	commands.Must(s.Cobra().MarkFlagRequired("name"))
 }
 
 // ExecuteSingleArgument implements commands.SingleArgumentCommand
diff --git a/internal/commands/server/create.go b/internal/commands/server/create.go
@@ -299,12 +299,12 @@ func (s *createCommand) InitCommand() {
 	// fs.BoolVar(&s.params.remoteAccess, "remote-access-enabled", def.remoteAccess, "Enables or disables the remote access.")
 	s.AddFlags(fs)
 
-	_ = s.Cobra().MarkFlagRequired("hostname")
-	_ = s.Cobra().MarkFlagRequired("zone")
+	commands.Must(s.Cobra().MarkFlagRequired("hostname"))
+	commands.Must(s.Cobra().MarkFlagRequired("zone"))
 }
 
 func (s *createCommand) InitCommandWithConfig(cfg *config.Config) {
-	_ = s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg))
+	commands.Must(s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg)))
 }
 
 // ExecuteWithoutArguments implements commands.NoArgumentCommand
diff --git a/internal/commands/server/firewall/create.go b/internal/commands/server/firewall/create.go
@@ -73,8 +73,8 @@ To edit the default rule of the firewall, set only ` + "`" + `--direction` + "`"
 	flagSet.StringVar(&s.comment, "comment", "", "Freeform comment that can include 0-250 characters.")
 
 	s.AddFlags(flagSet)
-	_ = s.Cobra().MarkFlagRequired("direction")
-	_ = s.Cobra().MarkFlagRequired("action")
+	commands.Must(s.Cobra().MarkFlagRequired("direction"))
+	commands.Must(s.Cobra().MarkFlagRequired("action"))
 	s.Cobra().MarkFlagsRequiredTogether("destination-port-start", "destination-port-end")
 	s.Cobra().MarkFlagsRequiredTogether("source-port-start", "source-port-end")
 }
diff --git a/internal/commands/server/firewall/delete.go b/internal/commands/server/firewall/delete.go
@@ -35,7 +35,7 @@ func (s *deleteCommand) InitCommand() {
 	flagSet.IntVar(&s.rulePosition, "position", 0, "Rule position. Available: 1-1000")
 	s.AddFlags(flagSet)
 
-	_ = s.Cobra().MarkFlagRequired("position")
+	commands.Must(s.Cobra().MarkFlagRequired("position"))
 }
 
 // Execute implements commands.MultipleArgumentCommand
diff --git a/internal/commands/server/load.go b/internal/commands/server/load.go
@@ -48,7 +48,7 @@ func (s *loadCommand) InitCommand() {
 
 	s.AddFlags(flagSet)
 
-	_ = s.Cobra().MarkFlagRequired("storage")
+	commands.Must(s.Cobra().MarkFlagRequired("storage"))
 }
 
 // Execute implements commands.MultipleArgumentCommand
diff --git a/internal/commands/server/networkinterface/create.go b/internal/commands/server/networkinterface/create.go
@@ -64,7 +64,7 @@ func (s *createCommand) InitCommand() {
 }
 
 func (s *createCommand) InitCommandWithConfig(cfg *config.Config) {
-	_ = s.Cobra().RegisterFlagCompletionFunc("network", namedargs.CompletionFunc(completion.Network{}, cfg))
+	commands.Must(s.Cobra().RegisterFlagCompletionFunc("network", namedargs.CompletionFunc(completion.Network{}, cfg)))
 }
 
 // MaximumExecutions implements Command.MaximumExecutions
diff --git a/internal/commands/server/networkinterface/delete.go b/internal/commands/server/networkinterface/delete.go
@@ -36,7 +36,7 @@ func (s *deleteCommand) InitCommand() {
 	fs.IntVar(&s.interfaceIndex, "index", 0, "Interface index.")
 
 	s.AddFlags(fs)
-	_ = s.Cobra().MarkFlagRequired("index")
+	commands.Must(s.Cobra().MarkFlagRequired("index"))
 }
 
 // MaximumExecutions implements Command.MaximumExecutions
diff --git a/internal/commands/server/networkinterface/modify.go b/internal/commands/server/networkinterface/modify.go
@@ -46,7 +46,7 @@ func (s *modifyCommand) InitCommand() {
 	fs.StringSliceVar(&s.ipAddresses, "ip-addresses", s.ipAddresses, "A comma-separated list of IP addresses, multiple can be declared\nUsage: --ip-address address=94.237.112.143,family=IPv4")
 
 	s.AddFlags(fs) // TODO(ana): replace usage with examples once the refactor is done.
-	_ = s.Cobra().MarkFlagRequired("index")
+	commands.Must(s.Cobra().MarkFlagRequired("index"))
 }
 
 // MaximumExecutions implements Command.MaximumExecutions
diff --git a/internal/commands/server/storage/attach.go b/internal/commands/server/storage/attach.go
@@ -58,7 +58,7 @@ func (s *attachCommand) InitCommand() {
 	config.AddToggleFlag(flagSet, &s.params.bootable, "boot-disk", false, "Set attached device as the server's boot disk.")
 
 	s.AddFlags(flagSet)
-	_ = s.Cobra().MarkFlagRequired("storage")
+	commands.Must(s.Cobra().MarkFlagRequired("storage"))
 }
 
 // MaximumExecutions implements command.Command
diff --git a/internal/commands/server/storage/detach.go b/internal/commands/server/storage/detach.go
@@ -47,7 +47,7 @@ func (s *detachCommand) InitCommand() {
 	flagSet.StringVar(&s.params.Address, "address", defaultDetachParams.Address, "Detach the storage attached to this address.")
 
 	s.AddFlags(flagSet)
-	_ = s.Cobra().MarkFlagRequired("address")
+	commands.Must(s.Cobra().MarkFlagRequired("address"))
 }
 
 // MaximumExecutions implements command.Command
diff --git a/internal/commands/servergroup/create.go b/internal/commands/servergroup/create.go
@@ -93,11 +93,11 @@ func (c *createCommand) InitCommand() {
 
 	c.AddFlags(fs)
 
-	_ = c.Cobra().MarkFlagRequired("title")
+	commands.Must(c.Cobra().MarkFlagRequired("title"))
 }
 
 func (c *createCommand) InitCommandWithConfig(cfg *config.Config) {
-	_ = c.Cobra().RegisterFlagCompletionFunc("server", namedargs.CompletionFunc(completion.Server{}, cfg))
+	commands.Must(c.Cobra().RegisterFlagCompletionFunc("server", namedargs.CompletionFunc(completion.Server{}, cfg)))
 }
 
 // ExecuteWithoutArguments implements commands.NoArgumentCommand
diff --git a/internal/commands/servergroup/modify.go b/internal/commands/servergroup/modify.go
@@ -88,6 +88,10 @@ func (c *modifyCommand) InitCommand() {
 	c.AddFlags(fs)
 }
 
+func (c *modifyCommand) InitCommandWithConfig(cfg *config.Config) {
+	commands.Must(c.Cobra().RegisterFlagCompletionFunc("server", namedargs.CompletionFunc(completion.Server{}, cfg)))
+}
+
 // Execute implements commands.MultipleArgumentCommand
 func (c *modifyCommand) Execute(exec commands.Executor, uuid string) (output.Output, error) {
 	svc := exec.All()
@@ -109,7 +113,3 @@ func (c *modifyCommand) Execute(exec commands.Executor, uuid string) (output.Out
 
 	return output.OnlyMarshaled{Value: res}, nil
 }
-
-func (c *modifyCommand) InitCommandWithConfig(cfg *config.Config) {
-	_ = c.Cobra().RegisterFlagCompletionFunc("server", namedargs.CompletionFunc(completion.Server{}, cfg))
-}
diff --git a/internal/commands/storage/backup/create.go b/internal/commands/storage/backup/create.go
@@ -48,7 +48,7 @@ func (s *createBackupCommand) InitCommand() {
 	flagSet.StringVar(&s.params.Title, "title", defaultCreateBackupParams.Title, "A short, informational description.")
 
 	s.AddFlags(flagSet)
-	_ = s.Cobra().MarkFlagRequired("title")
+	commands.Must(s.Cobra().MarkFlagRequired("title"))
 }
 
 // Execute implements commands.MultipleArgumentCommand
diff --git a/internal/commands/storage/clone.go b/internal/commands/storage/clone.go
@@ -57,12 +57,12 @@ func (s *cloneCommand) InitCommand() {
 	config.AddToggleFlag(flagSet, &s.params.encrypted, "encrypt", false, "Encrypt the new storage.")
 
 	s.AddFlags(flagSet)
-	_ = s.Cobra().MarkFlagRequired("title")
-	_ = s.Cobra().MarkFlagRequired("zone")
+	commands.Must(s.Cobra().MarkFlagRequired("title"))
+	commands.Must(s.Cobra().MarkFlagRequired("zone"))
 }
 
 func (s *cloneCommand) InitCommandWithConfig(cfg *config.Config) {
-	_ = s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg))
+	commands.Must(s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg)))
 }
 
 // Execute implements commands.MultipleArgumentCommand
diff --git a/internal/commands/storage/create.go b/internal/commands/storage/create.go
@@ -94,12 +94,12 @@ func (s *createCommand) InitCommand() {
 	applyCreateFlags(s.flagSet, &s.params, defaultCreateParams)
 
 	s.AddFlags(s.flagSet)
-	_ = s.Cobra().MarkFlagRequired("title")
-	_ = s.Cobra().MarkFlagRequired("zone")
+	commands.Must(s.Cobra().MarkFlagRequired("title"))
+	commands.Must(s.Cobra().MarkFlagRequired("zone"))
 }
 
 func (s *createCommand) InitCommandWithConfig(cfg *config.Config) {
-	_ = s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg))
+	commands.Must(s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg)))
 }
 
 // ExecuteWithoutArguments implements commands.NoArgumentCommand
diff --git a/internal/commands/storage/import.go b/internal/commands/storage/import.go
@@ -74,7 +74,7 @@ func (s *importCommand) InitCommand() {
 	applyCreateFlags(flagSet, &s.createParams, defaultCreateParams)
 
 	s.AddFlags(flagSet)
-	_ = s.Cobra().MarkFlagRequired("source-location")
+	commands.Must(s.Cobra().MarkFlagRequired("source-location"))
 }
 
 type storageImportStatus struct {
diff --git a/internal/commands/storage/templatize.go b/internal/commands/storage/templatize.go
@@ -54,7 +54,7 @@ func (s *templatizeCommand) InitCommand() {
 	config.AddToggleFlag(flagSet, &s.wait, "wait", false, "Wait for storage to be in online state before returning.")
 
 	s.AddFlags(flagSet)
-	_ = s.Cobra().MarkFlagRequired("title")
+	commands.Must(s.Cobra().MarkFlagRequired("title"))
 }
 
 // Execute implements commands.MultipleArgumentCommand
diff --git a/internal/commands/util.go b/internal/commands/util.go
@@ -135,3 +135,10 @@ func ParseSSHKeys(sshKeys []string) ([]string, error) {
 
 	return allSSHKeys, nil
 }
+
+// Must panics if the error is not nil.
+func Must(err error) {
+	if err != nil {
+		panic(err)
+	}
+}

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ func (l *listCommand) InitCommand() {`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`func (l listCommand) InitCommandWithConfig(cfg config.Config) {`
`40`		`- _ = l.Cobra().RegisterFlagCompletionFunc("username", namedargs.CompletionFunc(completion.Username{}, cfg))`
	`40`	`+ commands.Must(l.Cobra().RegisterFlagCompletionFunc("username", namedargs.CompletionFunc(completion.Username{}, cfg)))`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`func (l *listCommand) ExecuteWithoutArguments(exec commands.Executor) (output.Output, error) {`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ func (s *deleteCommand) InitCommand() {`
`36`	`36`	`flagSet.StringVar(&s.name, "name", "", "Index name")`
`37`	`37`	`s.AddFlags(flagSet)`
`38`	`38`
`39`		`- _ = s.Cobra().MarkFlagRequired("name")`
	`39`	`+ commands.Must(s.Cobra().MarkFlagRequired("name"))`
`40`	`40`	`}`
`41`	`41`
`42`	`42`	`// Execute implements commands.MultipleArgumentCommand`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ func (s *cancelCommand) InitCommand() {`
`41`	`41`	`config.AddToggleFlag(flagSet, &s.terminate, "terminate", false, "Request immediate termination instead of soft cancel.")`
`42`	`42`
`43`	`43`	`s.AddFlags(flagSet)`
`44`		`- _ = s.Cobra().MarkFlagRequired("pid")`
	`44`	`+ commands.Must(s.Cobra().MarkFlagRequired("pid"))`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`// Execute implements commands.MultipleArgumentCommand`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ func (s *assignCommand) InitCommand() {`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`func (s assignCommand) InitCommandWithConfig(cfg config.Config) {`
`60`		`- _ = s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg))`
	`60`	`+ commands.Must(s.Cobra().RegisterFlagCompletionFunc("zone", namedargs.CompletionFunc(completion.Zone{}, cfg)))`
`61`	`61`	`}`
`62`	`62`
`63`	`63`	`// ExecuteWithoutArguments implements commands.NoArgumentCommand`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ func (s *deleteCommand) InitCommand() {`
`35`	`35`	`flagSet.StringVar(&s.name, "name", "", "Node group name")`
`36`	`36`	`s.AddFlags(flagSet)`
`37`	`37`
`38`		`- _ = s.Cobra().MarkFlagRequired("name")`
	`38`	`+ commands.Must(s.Cobra().MarkFlagRequired("name"))`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`// ExecuteSingleArgument implements commands.SingleArgumentCommand`
Original file line number	Diff line number	Diff line change
`@@ -37,8 +37,8 @@ func (s *scaleCommand) InitCommand() {`
`37`	`37`	`flagSet.IntVar(&s.count, "count", 0, "Node count")`
`38`	`38`	`s.AddFlags(flagSet)`
`39`	`39`
`40`		`- _ = s.Cobra().MarkFlagRequired("name")`
`41`		`- _ = s.Cobra().MarkFlagRequired("count")`
	`40`	`+ commands.Must(s.Cobra().MarkFlagRequired("name"))`
	`41`	`+ commands.Must(s.Cobra().MarkFlagRequired("count"))`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`// ExecuteSingleArgument implements commands.SingleArgumentCommand`
Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ func (s *showCommand) InitCommand() {`
`37`	`37`	`flagSet.StringVar(&s.name, "name", "", "Node group name")`
`38`	`38`	`s.AddFlags(flagSet)`
`39`	`39`
`40`		`- _ = s.Cobra().MarkFlagRequired("name")`
	`40`	`+ commands.Must(s.Cobra().MarkFlagRequired("name"))`
`41`	`41`	`}`
`42`	`42`
`43`	`43`	`// ExecuteSingleArgument implements commands.SingleArgumentCommand`