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

Cannot run service commands containing a colon #186

Open
3 tasks done
Brewal opened this issue Aug 2, 2023 · 3 comments
Open
3 tasks done

Cannot run service commands containing a colon #186

Brewal opened this issue Aug 2, 2023 · 3 comments

Comments

@Brewal
Copy link

Brewal commented Aug 2, 2023

Are you certain it's a bug?

  • Yes, it looks like a bug

Are you using the latest version?

  • Yes, I'm using the latest version

Is there an existing issue for this?

  • I have searched existing issues, it hasn't been reported yet

Issue description

When trying to run a command with compose that contains a : character, it seems to be replaced with a space.
I tried to escape the colon, put the command between double quotes, ... with no luck.

Same goes with the shortcut alternative :

serverless api:bref:cli --args="list"

Service configuration (serverless-compose.yml) content

services:
  api:
    path: api

Command name and used flags

serverless bref:cli --service=api --args="list"

Command output

Error:
Serverless command "bref cli" not found. Did you mean "bref:cli"? Run "serverless help" for a list of all available commands.
@mnapoli
Copy link
Contributor

mnapoli commented Aug 2, 2023

@Brewal that is related to how Bref commands look (they follow the Symfony Console pattern so that they look more familiar to PHP devs).

Long story short, they are not supported by Serverless Compose because of that :.

@Brewal
Copy link
Author

Brewal commented Aug 2, 2023

Thanks for the response ! So I understand that this won't be fix and that there's no other workaround than cd into the corresponding service directory and use the regular serverless command ?

I'm a bit curious about why this is a limitation :)

@mnapoli
Copy link
Contributor

mnapoli commented Aug 7, 2023

I think so yes. One way to fix this would be to update Bref's repo to support both bref:cli and bref cli, that would be doable I think.

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

No branches or pull requests

2 participants