When passing empty -v in set goml does not return an error

[andrew-edgar]

If using goml set and the value is not specified the incorrect usage is shown but the return code is 0 which causes scripts to not exit.

Can all incorrect usage messages produce a non 0 error?

coligo goml set -f eprops-test/properties.yml -p "template.version" -v
Incorrect Usage: flag needs an argument: -v
NAME:
   goml set - Set/Update property
USAGE:
   goml set [command options] [arguments...]
OPTIONS:
   --file value, -f value   path to YAML file
   --prop value, -p value   property path string - foo.bar.zoo
   --value value, -v value  value for the defined property
   --key value, -k value    private key file
   --dry-run, -d            print set result to stdout
   --json, -j               format output as json
➜  coligo echo $?
0

[andrew-edgar]

I think you just need to check the error in cmd.Run() …

https://github.com/herrjulz/goml/blob/master/cmd/goml/main.go#L78

Should look more like this …

	err := cmd.Run(os.Args)
	if err != nil {
		log.Fatal(err)
	}