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
fix(node): wait for subprocess to exit when --watch=false
in execute executor
#9087
fix(node): wait for subprocess to exit when --watch=false
in execute executor
#9087
Conversation
…cute executor Fixes #7031. Before this fix, the executor would exit immediately and let the node subprocess continue in the background, if watch was set to false. This commit attempts to fix this problem, by adding event listeners on the subprocess in the node/execute executor if watch is false, such that the nx process continues simply (a)waits until the subprocess exists (successfully or not). Error handling has also been added such that the executor fails if a non-successful status code is returned by the program
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/nrwl/nx-dev/uwgHgyaDVP7dJKNu6Vewc8MsTFFf |
--watch=false
in execute executor
…t-when-watch-false
I think this may answer my question here: #5726 |
Is there anything preventing this from being merged and included in the next release? |
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Fixes #7031
Current Behavior
Before this fix, the node/execute executor would exit immediately and let the node subprocess continue in the background, if watch was set to false.
Expected Behavior
This PR attempts to fix this problem, by implementing the patch suggested in #7031 (comment), adding event listeners on the subprocess if watch is false, such that the nx process simply waits until the subprocess exists (successfully or not).
Error handling has also been added such that the executor fails if a non-successful status code is returned by the program