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

CI Pipeline does not receive proper exit codes #87

Closed
AndyOGo opened this issue Feb 15, 2017 · 3 comments
Closed

CI Pipeline does not receive proper exit codes #87

AndyOGo opened this issue Feb 15, 2017 · 3 comments

Comments

@AndyOGo
Copy link

AndyOGo commented Feb 15, 2017

First I wanna state that this is a great library.
Though I face the issue that if one of my build scripts breaks my CI is not "notified" though proper erring exit codes, instead my npm scripts just hangs and does nothing.
I expect that if my JS or CSS or anything breaks, that all other scripts can be cancelled and that the parent process exits which non-zero code.
May I miss some CLI option?

I use npm-run-all v^4.0.1 in combination with better-npm-run v0.0.11

@AndyOGo
Copy link
Author

AndyOGo commented Feb 16, 2017

I just tried same setup with cross-env, unfortunately with same results 😞

@AndyOGo
Copy link
Author

AndyOGo commented Feb 16, 2017

I further investigated this issue and surprisingly found that it does not seem to be an issue with npm-run-all at all.

Turned out that:

To double check that npm-run-all does things right, wrote some small test scripts which throw at various posistions, namely first-line, within setImmediate callback and within setTimeout callback. All of them worked perfectly...

@AndyOGo
Copy link
Author

AndyOGo commented Feb 17, 2017

@mysticatea
Sorry for my wrong bug report.
I could solve all issues by fixing/workaround above issues ... will close.

@AndyOGo AndyOGo closed this as completed Feb 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants