- Notifications
You must be signed in to change notification settings - Fork198
Added support for terminating a spawned worker if a timeout is reached#207
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Epic. Please also add a description of the new option in the readme file too.
Thanks! I've added an update to the documentation. However, is it possible that the new timeout option will overwrite an environment variable that could be set in the Parallel constructor? |
Thank you! |
released as part of v1.1.0 |
Hi, I found this lib because I was searching for a way to easily spawn a worker, but also be able to terminate it if a timeout is reached. The use case is that I want to time limit execution for certain operations.
I don't have any experience with the test frameworks in this lib, but if my pull request looks promising, I'm happy to learn and create a test spec. But only if you think this extra feature even belongs in parallel.js?