Skip unknown options check if there is a better error to display

[shadowspawn]

Pull Request

I think this is working our sensibly. Making changes in a complicated piece of the code so going slowly.

Problem

In Commander v5 we started checking for unknown options in more situations, which was an improvement in many cases. However, it unintentional blocked some "better" errors such as an unknown command, and blocked suggestions for the unknown commands (listener for command:*).

Related issue: #1460

Solution

Work through cases and don't block a better error by checking for unknown options. Especially, when the issue is likely to be a misspelled command.

ChangeLog

• fix regression for legacy command('*') and call when command line includes options (Skip unknown options check if there is a better error to display #1464)
• fix regression for on('command:*', ... and call when command line includes unknown options (Skip unknown options check if there is a better error to display #1464)
• fix display best error for combination of unknown command and unknown option (i.e. unknown command) (Skip unknown options check if there is a better error to display #1464)

[abetomo]

Nice!