CLI is silent when passed an unrecognized option

[sedubois]

Describe the bug
The lingui CLI is silent when it is passed an unrecognized option.

To Reproduce
$ lingui foo

Expected behavior
The CLI should display an error.

lingui --version: 2.4.1

[tricoder42]

Indeed, this is how commander works, but it should at least show a help. Here's description of issue and possible fix: tj/commander.js#432 (comment)

Interested in sending PR?

[sedubois]

I won't have time to send a PR but the issue is not critical so feel free to close, I just wanted to make it known.

[tricoder42]

Let’s leave it open as a known bug. Maybe someone will pick it up ;)

…

On 22 Aug 2018, at 09:36, Sébastien Dubois ***@***.***> wrote: I won't have time to send a PR but the issue is not critical so feel free to close, I just wanted to make it known. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or mute the thread.

[dephiros]

@tricoder42 , it seems like something that I can contribute(trying to familiarize myself more with the project)
Will try to look into it and give a pull request soon!

[tricoder42]

@dephiros That would be awesome! See the linked comment, there're some possible solutions.

[dephiros]

@tricoder42 , should we address this with just return error or try to be more fancy with fuzzy match and return closest subcommands? With the fuzzy match, that would require an additional lib like hamming-compute unless we already have an alternative

[tricoder42]

Sounds interesting! It's a backend code so I don't mind extra dependency. Feel free to do it if you're interested 👍

[dephiros]

@tricoder42 , I am almost done with the pull request but stuck at a strange unit test failure: https://circleci.com/gh/lingui/js-lingui/2224?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
Could you help me take a look when you have a chance?
Thank you

[danielkcz]

@dephiros I think you don't need to worry about that one, I've seen that failing before as well. There is something broken in general it seems. Just hold tight and @tricoder42 will surely merge it soon.

[tricoder42]

Released in v2.7.0. Thanks @dephiros!