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

Add pass arguments support #282

wants to merge 9 commits into from


Copy link

@loynoir loynoir commented Jun 4, 2021



  1. add prefix string to each command
  2. add suffix string to each command


Let fd/find feed files arguments to concurrently, instead of feed command arguments.

fd test.mjs directories \
    -X concurrently --kill-others-on-fail \
    -P node
concurrently --kill-others-on-fail \
    'node a1/test.mjs' \
    'node a2/test.mjs' \
    'node a3/test.mjs' \
    'node a4/test.mjs' \
    'node a5/test.mjs' \
    'node a6/test.mjs' \
    'node a7/test.mjs' \
fd serve.mjs directories \
    -X concurrently \
    -P node \
    -D append='--watch'
concurrently \
    'node a1/serve.mjs --watch' \
    'node a2/serve.mjs --watch' \
    'node a3/serve.mjs --watch' \
    'node a4/serve.mjs --watch' \
    'node a5/serve.mjs --watch' \
    'node a6/serve.mjs --watch' \
    'node a7/serve.mjs --watch' \

@loynoir loynoir changed the title Pass arguments support Add pass arguments support Jun 4, 2021
Copy link

@coveralls coveralls commented Jun 4, 2021

Coverage Status

Coverage increased (+0.01%) to 99.762% when pulling 32e3a84 on loynoir:master into 10ff00c on kimmobrunfeldt:master.

loynoir added 5 commits Jun 4, 2021
Using `-D append='-- --watch'` style,
instead of `--append ' -- --watch'`
to prevent yargs miss parse dash.
Copy link

@paescuj paescuj commented Nov 30, 2021

Would be great to have this released soon 👍 Is there anything I can help with?

Copy link

@loynoir loynoir commented Dec 1, 2021


Copy link

@gustavohenke gustavohenke left a comment

Sorry that I never got to review this!
It'd need to be updated to match the current codebase language before it can be merged.

There's also #307 which seems to be achieving the same.

@paescuj paescuj closed this in #307 May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet

Successfully merging this pull request may close these issues.

None yet

4 participants