Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unhandled exception when --target has unexpected value #735

Closed
3 tasks done
Ky6uk opened this issue Sep 11, 2020 · 5 comments
Closed
3 tasks done

Unhandled exception when --target has unexpected value #735

Ky6uk opened this issue Sep 11, 2020 · 5 comments

Comments

@Ky6uk
Copy link

Ky6uk commented Sep 11, 2020

  • I have searched for similar issues
  • I am using the latest version of npm-check-updates
  • I am using node >= 10.17

ncu --target woof will throw an error with the stack trace to the output channel. I'm not sure, but it feels like something unhandled.

@raineorshine
Copy link
Owner

Yeah, we can do a nicer error message there. I'm going to wait until commander.js v7 when the choices feature is added.

tj/commander.js#1331

@LitoMore
Copy link
Contributor

I strongly recommend you to use meow. For using meow, we don't need to wait any feature from upstream, because it's simple and powerful.

@raineorshine
Copy link
Owner

I strongly recommend you to use meow.

Looks great, but I don't think it's worth switching cli frameworks just for validation, which can be easily implemented if we don't want to wait for commander.

For using meow, we don't need to wait any feature from upstream, because it's simple and powerful.

Actually, we're waiting for help text generation.

@LitoMore
Copy link
Contributor

Actually, we're waiting for help text generation.

I can help. ✌️

@raineorshine
Copy link
Owner

This was done at some point

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants