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

Remove strict requirement for -l debug and -o flags from twilio-cli #202

Closed
deathemperor opened this issue Jul 21, 2020 · 7 comments
Closed
Labels
status: help wanted requesting help from the community triage queue Issue is in our internal backlog. It's either a bug or a feature enhancement. type: twilio enhancement feature request on Twilio's roadmap

Comments

@deathemperor
Copy link

Issue Summary

Getting an unknown error when initializing a serverless project using verify template. Then unable to add -l debug to twilio-cli

Steps to Reproduce

  1. Install twilio-cli
  2. run twilio serverless:init verify-sample --template=verify
  3. run twilio serverless:init verify-sample --template=verify -l debug

Technical details:

  • twilio-cli version: 2.5.0
  • node version: 12.10.0
  • Command output with debug logging enabled (adding -l debug to the end of the command): unable to run with -l debug

Screen Shot 2020-07-21 at 3 29 50 PM

@philnash
Copy link
Contributor

philnash commented Jul 22, 2020

Hey there! The serverless command is provided by the @twilio-labs/plugin-serverless plugin and ultimately calls @twilio-labs/create-twilio-function which I maintain and has not implemented the -l flag yet (issue has been raised for this here).

I am concerned about the error though. There shouldn't be an issue between the plugin and the CLI.

Can you run twilio plugins and tell me which version of @twilio-labs/plugin-serverless you are using?

Are you able to run the same command with create-twilio-function?

npm init twilio-function verify-sample --template=verify

Thanks for reporting this.

@deathemperor
Copy link
Author

Can you run twilio plugins and tell me which version of @twilio-labs/plugin-serverless you are using?

Runnning twillio plugins I only get this (no version):

twilio plugins
@twilio-labs/plugin-serverless

but I've just reinstalled the plugin so is it safe to assume it's the latest?

Are you able to run the same command with create-twilio-function?

I'm able to create the verify-sample project with this command.

Thanks for your support @philnash .

@philnash
Copy link
Contributor

Thanks for following up @deathemperor. I have some more questions as I am trying to reproduce this error.

Running twilio plugins:update should give you the latest version of the serverless plugin, which should be 1.7.1.

Do you have a profile set up for the Twilio CLI? (Did you call twilio login or twilio profiles:create at some point?)

Do the other twilio serverless commands work for you? You should be able to run them within the project you have now been able to create.

@deathemperor
Copy link
Author

Do you have a profile set up for the Twilio CLI? (Did you call twilio login or twilio profiles:create at some point?)

yes I do.

Do the other twilio serverless commands work for you? You should be able to run them within the project you have now been able to create.

yes it does, like twilio serverless:start. In fact, I've always been able to run twilio serverless:start and twilio serverless:deploy and always failed to run the init. But back then I create the twilio project myself and then add the verify js files. Yesterday I tried doing the verify template again (there was some issue with the serverless deployment) to find a fix and found this. FYI my node version is 12.

@eshanholtz eshanholtz added status: help wanted requesting help from the community type: twilio enhancement feature request on Twilio's roadmap labels Jul 24, 2020
@eshanholtz
Copy link
Contributor

eshanholtz commented Jul 24, 2020

It looks like not all plugins are supporting the -l and -o flags. The issue to fix the messaging has been added to our internal backlog to be prioritized (reference ID: DI-810). Pull requests and +1s on the issue summary will help it move up the backlog.

@eshanholtz
Copy link
Contributor

@deathemperor Forgot to mention, for further help with the serverless plugin, please open a separate issue in the plugin-serverless github repo

@eshanholtz eshanholtz changed the title Add -l debug to twilio-cli, get 'Error: Unexpected arguments: -l, debug' Remove strict requirement for -l debug and -o flags from twilio-cli Jul 31, 2020
@shrutiburman shrutiburman added the triage queue Issue is in our internal backlog. It's either a bug or a feature enhancement. label Nov 15, 2022
@bnb
Copy link
Member

bnb commented Oct 15, 2024

since this is for a plugin that's currently publicly archived and there's been no additional feedback from the original issue author, I'm going to go ahead and close it. Please fee free to either let us know here if this is still an issue that's relevant to the core twilio CLI, or feel free to create another issue referencing this one.

@bnb bnb closed this as completed Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: help wanted requesting help from the community triage queue Issue is in our internal backlog. It's either a bug or a feature enhancement. type: twilio enhancement feature request on Twilio's roadmap
Projects
None yet
Development

No branches or pull requests

5 participants