Fixes error handling when an invalid flag is passed.

[jonsmorrow]

Signed-off-by: Jon Morrow jmorrow@chef.io

[marcparadise]

Looks good, just a minor wording change.

[marcparadise]

This \ gets pretty long (especially with upcoming ssh flags), making it likely someone has to scroll up to see what they did wrong.
Can we do something like:

The flag provided does not exist. 

Available flags are: 
  %2
You gave: 
  %1 

[jonsmorrow]

I copied the format for missing commands.

[marcparadise]

That may need a separate PR, though I think the size of the dynamic portion isn't so big there

[marcparadise]

Finishing the thought: . I'm +1 on landing this as it is for now - I'm planning to take a final pass through all of en.yml and check spelling, grammar, verboseness, word wrapping v term size, consistency, etc and will make tweaks as needed then.

[tyler-ball]

I feel like we should only be making changes to chef-run and deal with the whole chef-cli duplication separately. But these changes look good

[marcparadise]

@tyler-ball yeah, I have not been duplicating my changes into chef-cli for other things.

[jonsmorrow]

i did it on this only because it is likely handling we will need later. but in general yes we should stay in chef-run.

[jonsmorrow]

I am going to merge this since it fixes an error condition and matches existing ux style. I'll send a link to @vebberts and we can update in another pass if needed.

[jonsmorrow]

➜  chef-run git:(jm/fix_invalid_flag_handling) bundle exec bin/chef-run -g

The flag '-g' does not exist.

  Available flags are:
    -c, --config PATH                 Location of config file. Defaults to /Users/jmorrow/.chef-workstation/config.toml
        --cookbook-repo-paths PATH    Comma separated list of cookbook repository paths.
    -h, --help                        Show help and usage for `chef-run`
    -i, --identity-file PATH          SSH identity file to use when connecting.
        --[no-]install                Install Chef client on the target host(s) if it is not installed.
                                      This defaults to enabled - the installation will be performed
                                      if there is no Chef client on the target(s).
        --[no-]root                   Whether to use root permissions on the target. Defaults to true.
    -s, --[no-]ssl                    Use SSL for WinRM. Current default: false
    -s, --[no-]ssl-verify             Verify peer certificate when using SSL for WinRM
                                      Use --ssl-no-verify when using SSL for WinRM and
                                      the remote host is using a self-signed certificate.
                                      Current default: true
    -v, --version                     Show the current version of Chef Run.