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

camelCase argument in command requires inferred alias #1246

Closed
vasco-santos opened this issue Nov 19, 2018 · 0 comments · Fixed by #1247
Closed

camelCase argument in command requires inferred alias #1246

vasco-santos opened this issue Nov 19, 2018 · 0 comments · Fixed by #1247

Comments

@vasco-santos
Copy link

After yargs/yargs-parser/commit/08c011796f83e84fe50ffb066a87ae2effadfc75 got merged, commands using camelCase parameters were automatically inferred to have a dash case alias.

When we use this feature combined with using ... to optionally accept a list, throws an error Unknow argument, for instance in ipfs/js-ipfs/src/cli/commands/pin/ls.js#L7

In order to fix this, we have to set a dash-case option. This seems a bug with yargs-parser, as if it inferres other alias, it should be transparent to the users.

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

Successfully merging a pull request may close this issue.

1 participant