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

Same tasks are skipped #12

Closed
catamphetamine opened this issue Nov 4, 2015 · 4 comments
Closed

Same tasks are skipped #12

catamphetamine opened this issue Nov 4, 2015 · 4 comments
Assignees
Labels

Comments

@catamphetamine
Copy link

My script is

npm-run-all delay delay delay development-web-server
"delay": "node ./code/delay 3000"

But it only delays for 3 seconds and then runs development-web-server.

> webapp@1.0.0 development-web-server-delayed G:\work\webapp
> npm-run-all delay delay delay development-web-server


> webapp@1.0.0 delay G:\work\webapp
> node ./code/delay 3000


> webapp@1.0.0 development-web-server G:\work\webapp
> nodemon ./code/web-server/entry.js
@catamphetamine
Copy link
Author

My workaround for those who google - duplicating tasks
remy/nodemon#709 (comment)

@mysticatea
Copy link
Owner

Thank you for reporting.

npm-run-all is removing duplication for when glob-like patterns are overlapped.
But I think it should not remove duplication of non glob-like patterns.
I will fix it.

By the way, npm-run-all can use arguments, so that can be written:

{
    "development-web-server-delayed": "npm-run-all 'delay 9000' development-web-server",
    "delay": "node ./code/delay"
}

@mysticatea mysticatea self-assigned this Nov 4, 2015
@mysticatea mysticatea added the bug label Nov 4, 2015
@catamphetamine
Copy link
Author

@mysticatea oh, the arguments feature is good, i'll use it in my code instead, thanks

@catamphetamine
Copy link
Author

@mysticatea thx for the fix. It works now with duplicates.

The initial suggestion didn't work

{
    "development-web-server-delayed": "npm-run-all 'delay 9000' development-web-server",
    "delay": "node ./code/delay"
}

(the delay script didn't even run)
but it doesn't matter since duplicates are working now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants