Skip to content

Commit

Permalink
make guarantees about orphaned processes
Browse files Browse the repository at this point in the history
- updates workerpool to v6.0.2, which guarantees child processes exit before `Pool#terminate()` resolves
- cleanup `test/integration/options/parallel.spec.js`
  • Loading branch information
boneskull committed Oct 9, 2020
1 parent f24f190 commit 6ceca82
Show file tree
Hide file tree
Showing 4 changed files with 369 additions and 291 deletions.
4 changes: 1 addition & 3 deletions lib/nodejs/buffered-worker-pool.js
Expand Up @@ -75,9 +75,7 @@ class BufferedWorkerPool {
process.execArgv.join(' ')
);

this.options = Object.assign({}, WORKER_POOL_DEFAULT_OPTS, opts, {
maxWorkers
});
this.options = {...WORKER_POOL_DEFAULT_OPTS, opts, maxWorkers};
this._pool = workerpool.pool(WORKER_PATH, this.options);
}

Expand Down
12 changes: 9 additions & 3 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 2 additions & 1 deletion package.json
Expand Up @@ -73,7 +73,7 @@
"supports-color": "7.1.0",
"which": "2.0.2",
"wide-align": "1.1.3",
"workerpool": "6.0.0",
"workerpool": "6.0.2",
"yargs": "13.3.2",
"yargs-parser": "13.1.2",
"yargs-unparser": "1.6.1"
Expand Down Expand Up @@ -130,6 +130,7 @@
"needle": "^2.5.0",
"nps": "^5.10.0",
"nyc": "^15.1.0",
"pidtree": "^0.5.0",
"prettier": "^1.19.1",
"remark": "^12.0.1",
"remark-github": "^9.0.1",
Expand Down

0 comments on commit 6ceca82

Please sign in to comment.