Separate mandatory from non-mandatory flags

daisy · Feb 5, 2019 · c39f7e3 · c39f7e3
1 parent d68b524
commit c39f7e3
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/cli/cli.go b/cli/cli.go
@@ -46,16 +46,17 @@ Admin commands:
 List of global options:                 {{.Name}} help -g 
 Detailed help for a single command:     {{.Name}} help COMMAND
 `
-	//TODO: Check if required options to write/ignore []
 	COMMAND_HELP_TEMPLATE = `
-Usage: {{.Parent.Name}} [GLOBAL_OPTIONS] {{.Name}} [OPTIONS]  {{ .Arity.Description}}
+Usage: {{.Parent.Name}} [GLOBAL_OPTIONS] {{.Name}}{{if .MandatoryFlags}} REQUIRED_OPTIONS{{end}}{{if .MandatoryFlags}} [OPTIONS]{{end}} {{ .Arity.Description}}
 
 {{.Description}}
-{{if .Flags}}
-Options:
-{{range .Flags }}       {{flagAligner .FlagStringPrefix}} {{.ShortDesc}}
-{{end}}
-{{end}}
+
+{{if .MandatoryFlags}}Required options:
+{{range .MandatoryFlags }}       {{flagAligner .FlagStringPrefix}} {{.ShortDesc}}
+{{end}}{{end}}
+{{if .NonMandatoryFlags}}{{if .MandatoryFlags}}Other options{{end}}{{if not .MandatoryFlags}}Options{{end}}:
+{{range .NonMandatoryFlags }}       {{flagAligner .FlagStringPrefix}} {{.ShortDesc}}
+{{end}}{{end}}
 
 `
 

diff --git a/cli/scripts.go b/cli/scripts.go
@@ -192,6 +192,7 @@ func scriptToCommand(script pipeline.Script, cli *Cli, link *PipelineLink) (req
 		}
 		return nil
 	}, jobRequest)
+	command.SetArity(0, "")
 
 	for _, input := range script.Inputs {
 		name := getFlagName(input.Name, "i-", command.Flags())