Try removing lerna hoist, node_modules cache, and parallel tests

deephaven · Apr 29, 2021 · 90d1c21 · 90d1c21
1 parent 50e5c8f
commit 90d1c21
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 12 deletions.
diff --git a/.github/workflows/node.js.yml b/.github/workflows/node.js.yml
@@ -25,14 +25,6 @@ jobs:
         with:
           node-version: ${{ matrix.node-version }}
 
-      - name: Cache node_modules
-        id: cache-node-modules
-        uses: actions/cache@v2
-        with:
-          # If no package-locks have changed, it should be safe to restore all node_modules as they were
-          path: '**/node_modules'
-          key: ${{ runner.os }}-modules-${{ hashFiles('**/package-lock.json') }}
-
       - name: Cache npm cache
         uses: actions/cache@v2
         env:
@@ -56,9 +48,7 @@ jobs:
             ${{ runner.os }}-eslintcache-
             ${{ runner.os }}-
 
-      # Only need to install and bootstrap deps if package-locks changed
       - name: Install dependencies
-        if: steps.cache-node-modules.outputs.cache-hit != 'true'
         run: npm ci
 
       - name: Test

diff --git a/package.json b/package.json
@@ -10,7 +10,7 @@
     "clean:build": "lerna run clean --stream",
     "clean:modules": "lerna clean --yes",
     "clean": "npm run clean:build && npm run clean:modules",
-    "bootstrap": "lerna bootstrap --hoist",
+    "bootstrap": "lerna bootstrap",
     "build": "lerna run build --stream",
     "build:app": "lerna run --scope=@deephaven/code-studio build",
     "build:packages": "lerna run --ignore=@deephaven/code-studio build --stream",
@@ -21,7 +21,7 @@
     "prestart": "lerna run prestart --stream",
     "pretest:ci": "npm run build:packages",
     "test": "lerna run test --stream --parallel",
-    "test:ci": "lerna run test:ci --stream --parallel",
+    "test:ci": "lerna run test:ci --stream",
     "sync-version": "lerna version ${DEEPHAVEN_VERSION:-error} --no-git-tag-version --yes",
     "publish": "lerna publish"
   },