fix(ci): clean up numerous disparate installs (#292)

WFCD · Feb 29, 2024 · 2307fe4 · 2307fe4
1 parent 879252c
commit 2307fe4
Show file tree

Hide file tree

Showing 3 changed files with 38 additions and 17 deletions.
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -6,25 +6,44 @@ on:
       - main
 
 jobs:
+  install:
+    name: Install
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v4
+    - uses: actions/setup-node@v4
+      with:
+        node-version-file: .nvmrc
+    - run: |
+       npm ci --omit=optional
+       npm install --no-save warframe-worldstate-parser@^3.x warframe-worldstate-data@^2.x
+    - uses: actions/cache/save@v4
+      with:
+        path: |
+          ./node_modules/**/*
+        key: node-ci-${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
   lint:
+    needs: install
     name: Lint
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v4
-    - name: Use Node.js ${{ matrix.node-version }}
+    - name: Use Node.js
       uses: actions/setup-node@v4
       with:
         node-version-file: .nvmrc
-    - name: Install Dependencies
-      run: |
-        npm ci --omit=optional
-        npm install --no-save warframe-worldstate-parser@^3.x warframe-worldstate-data@^2.x
+    - uses: actions/cache/restore@v4
+      with:
+        path: |
+          ./node_modules/**/*
+        key: node-ci-${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
     - name: Run linters
       run: npm run lint
   test:
     name: Test
     runs-on: ubuntu-latest
-    needs: lint
+    needs: [lint]
     strategy:
       matrix:
         node-version: ['lts/*', '20', '18']
@@ -34,27 +53,29 @@ jobs:
       uses: actions/setup-node@v4
       with:
         node-version: ${{ matrix.node-version }}
-    - name: Install Dependencies
-      run: |
-        npm ci --omit=optional
-        npm install --no-save warframe-worldstate-parser@^3.x warframe-worldstate-data@^2.x
+    - uses: actions/cache/restore@v4
+      with:
+        path: |
+          ./node_modules/**/*
+        key: node-ci-${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
     - run: npm test
       env:
         CI: true
         parallel: true
   coverage:
     name: Coverage
     runs-on: ubuntu-latest
-    needs: lint
+    needs: [lint, install]
     steps:
     - uses: actions/checkout@v4
     - uses: actions/setup-node@v4
       with:
         node-version-file: .nvmrc
-    - name: Install Dependencies
-      run: |
-        npm ci --omit=optional
-        npm install --no-save warframe-worldstate-parser@^3.x warframe-worldstate-data@^2.x
+    - uses: actions/cache/restore@v4
+      with:
+        path: |
+          ./node_modules/**/*
+        key: node-ci-${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
     - run: npm test
       env:
         CI: true

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -39,7 +39,7 @@
     "twitter": "^1.7.1"
   },
   "devDependencies": {
-    "@wfcd/eslint-config": "^1.3.3",
+    "@wfcd/eslint-config": "latest",
     "c8": "^9.1.0",
     "chai": "^4.3.4",
     "coveralls": "^3.1.0",