Skip to content

Commit 352922a

Browse files
committed
fix: use --expose-gc for top-level 'yarn test'
I added this to SwingSet/package.json in commit d4bc617 (PR #3136). However I overlooked the fact that #3100 had already landed, which changed the top-level `yarn test` from one that just did a (sequential) `yarn workspaces test`, testing one package at a time with whatever the local `package.json` said to do, to one that does a (parallel) `ava` invocation, which does all packages at the same time but ignores the local `package.json` settings. We need to include `--expose-gc` in the `nodeArguments` in the top-level package.json to make sure the parallel form give the correct arguments when SwingSet's turn comes around (as well as other packages which benefit from enabling GC but don't have tests which directly assert that it can be done).
1 parent 74869fe commit 352922a

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,9 @@
8787
"files": [
8888
"packages/*/test/**/test-*.js"
8989
],
90+
"nodeArguments": [
91+
"--expose-gc"
92+
],
9093
"require": [
9194
"esm"
9295
],

0 commit comments

Comments
 (0)