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

Respect CLI parameter order, and allow parameter naming #119

Open
oceansolutionsfr opened this issue Sep 15, 2022 · 0 comments
Open

Respect CLI parameter order, and allow parameter naming #119

oceansolutionsfr opened this issue Sep 15, 2022 · 0 comments

Comments

@oceansolutionsfr
Copy link

oceansolutionsfr commented Sep 15, 2022

with the following run cmd in script.yaml:
cmd: deno run script.ts --param1=$1 --param2=$2

currently, running vr run hello world is producing a shell command (in GNU bash, version 3.2.57) equivalent to deno run script.ts --param1= --param2= hello world but the expected shell command equivalent is deno run script.ts --param1=hello --param2=world

The feature should trim the parameters, and place them according to their respective positions in the script.yaml cmd line, and provide a way to name CLI parameters and to use references to the names within the cmd of the script.yaml (equivalent of npm ${npm_config_})

If it is already implemented, then maybe the docs should be updated with extended information about that (I can contribute if needed)

Best

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

1 participant