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

Closed
wants to merge 9 commits into from
Closed

Conversation

loynoir
Copy link

@loynoir loynoir commented Jun 4, 2021

issue33

Feature:

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

Examples

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
@coveralls
Copy link

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.

Using `-D append='-- --watch'` style,
instead of `--append ' -- --watch'`
to prevent yargs miss parse dash.
@paescuj
Copy link
Collaborator

paescuj commented Nov 30, 2021

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

@loynoir
Copy link
Author

loynoir commented Dec 1, 2021

@paescuj

Copy link
Member

@gustavohenke gustavohenke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants