-
Notifications
You must be signed in to change notification settings - Fork 94
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
Upload command from docs not working #176
Comments
@robertorubioguardia hi ! Thanks for reporting this. It's a known issue, when the
I will update the docs with your suggestion to avoid confusion in the future 😄 |
I just updated the docs. I'll close, but we'll reopen if further help is needed 😄 |
@gempain hello, is it a problem that branch can be an array (https://github.com/getmeli/meli-cli/blob/1ea83325797010c257c238536cccb4c653b04824/src/commands/upload/upload-options.ts#L49)? See the following link: yargs/yargs#164 So I think that's why the error comes, that one argument is missing. |
@phillipplum hi there ! Thanks for jumping 😄 The problem definitely comes from We could open an issue there to see what's going on. |
Hi guys,
I had to reorder the arguments in the upload command for it to work. Using it as stated in your docs it failed.
The working form was:
Note the folder was moved and put before the rest of the arguments. Error was:
Not enough non-option arguments: got 0, need at least 1
Cheers and thanks for such a great tool.
The text was updated successfully, but these errors were encountered: