status-im
/
metro
mirror of https://github.com/status-im/metro.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Pass numWorkers instead of maxWorkers to jest-worker 

Summary:
<!-- Thanks for submitting a pull request! Please provide enough information so that others can review your pull request. The two fields below are mandatory. -->

**Summary**

Jest Worker accepts parameter `numWorkers`, not `maxWorkers`, see:
https://github.com/facebook/jest/tree/master/packages/jest-worker#numworkers-number-optional

<!-- Explain the **motivation** for making this change. What existing problem does the pull request solve? -->

**Test plan**

<!-- Demonstrate the code is solid. Example: The exact commands you ran and their output, screenshots / videos if the pull request changes UI. -->
Closes https://github.com/facebook/metro/pull/136

Reviewed By: mjesun

Differential Revision: D7036411

Pulled By: rafeca

fbshipit-source-id: 5528ca4a9abe5a8e3668e83b214e10d97ba7a827
This commit is contained in:
Michal Glaus 2018-02-22 08:25:53 -08:00 committed by Facebook Github Bot
parent a984719eef
commit 40c2449eec
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
packages/metro/src/JSTransformer/index.js
View File

@ -139,7 +139,7 @@ module.exports = class Transformer {
}
}
_makeFarm(workerPath, computeWorkerKey, exposedMethods, maxWorkers) {
_makeFarm(workerPath, computeWorkerKey, exposedMethods, numWorkers) {
// We whitelist only what would work. For example `--inspect` doesn't work
// in the workers because it tries to open the same debugging port. Feel
// free to add more cases to the RegExp. A whitelist is preferred, to
@ -155,7 +155,7 @@ module.exports = class Transformer {
computeWorkerKey,
exposedMethods,
forkOptions: {execArgv},
maxWorkers,
numWorkers,
});
}