[urfavecli] handle nil required

Author: lispyclouds

README.md

@@ -18,7 +18,7 @@ climate allows the server to influence the CLI behaviour by using OpenAPI's [ext
 
 Overall, the way it works:
 
-- Each operation is converted to a Cobra command
+- Each operation is converted to a Cobra or urfave/cli command
 - Each parameter is converted to a flag with its corresponding type
 - As of now, request bodies are a flag and treated as a string regardless of MIME type. Name defaults to `climate-data` unless specified via `x-cli-name`. All subject to change
 - The provided handlers are attached to each command, grouped and attached to the rootCmd

urfavecli.go

@@ -32,31 +32,37 @@ func addParamsUrfaveCli(cmd *cli.Command, op *v3.Operation, handlerData *Handler
 
 	for _, param := range op.Parameters {
 		t := getParamType(param, op)
+		name := param.Name
+		desc := param.Description
+		required := false
+		if req := param.Required; req != nil {
+			required = *req
+		}
 
 		switch t {
 		case String:
 			flags = append(flags, &cli.StringFlag{
-				Name:     param.Name,
-				Usage:    param.Description,
-				Required: *param.Required,
+				Name:     name,
+				Usage:    desc,
+				Required: required,
 			})
 		case Integer:
 			flags = append(flags, &cli.IntFlag{
-				Name:     param.Name,
-				Usage:    param.Description,
-				Required: *param.Required,
+				Name:     name,
+				Usage:    desc,
+				Required: required,
 			})
 		case Number:
 			flags = append(flags, &cli.Float64Flag{
-				Name:     param.Name,
-				Usage:    param.Description,
-				Required: *param.Required,
+				Name:     name,
+				Usage:    desc,
+				Required: required,
 			})
 		case Boolean:
 			flags = append(flags, &cli.BoolFlag{
-				Name:     param.Name,
-				Usage:    param.Description,
-				Required: *param.Required,
+				Name:     name,
+				Usage:    desc,
+				Required: required,
 			})
 		default:
 			// TODO: array, object