Skip to content

Commit a5d8837

Browse files
authored
Use --concurrency 4 instead of --parallel for tests. (#1693)
This should make travis more reliable by limiting the amount of concurrency for our tests . `--concurrency 4` was also slightly faster than `--parallel` on my local machine. Though any value above 1 was actually fairly comparable: --parallel: 342.63s --concurrency 1: 498.26s --concurrency 2: 351.23s --concurrency 3: 342.50s --concurrency 4: 336.55s --concurrency 5: 349.82s --concurrency 6: 348.19s --concurrency 7: 341.47s --concurrency 8: 348.52s Comparing a couple travis runs from before (with `--parallel`) to this PR (with `--concurrency 4`), it seems like this speeds things up a bit: Before: 679.74s / 678.04s After: 595.03s / 635.62s And I notice the RTDB emulator starts muuuch faster: Before: Emulator has started up after 67.517s! After: Emulator has started up after 9.068s!
1 parent b27f4c7 commit a5d8837

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
"repl": "node tools/repl.js",
2525
"release": "node scripts/release/cli.js",
2626
"pretest": "node tools/pretest.js",
27-
"test": "lerna run --parallel --stream test",
27+
"test": "lerna run --concurrency 4 --stream test",
2828
"pretest:coverage": "mkdirp coverage",
2929
"test:coverage": "lcov-result-merger 'packages/**/lcov.info' | coveralls",
3030
"test:setup": "node tools/config.js",

0 commit comments

Comments
 (0)