Use composite action to set up node

.github/actions/prepare-node/action.yml

@@ -0,0 +1,39 @@
+name: Prepare Node
+description: Load npm cache, install Node dependencies
+
+inputs:
+  install-deps:
+    description: Whether to run `npm ci`. Set "maybe" to install deps if cache is missing. "yes" by default.
+    default: "yes"
+
+  ignore-scripts:
+    description: Whether to run `npm ci` with --ignore-scripts. "yes" by default.
+    default: "yes"
+
+runs:
+  using: composite
+  steps:
+    # Log debug information
+    - shell: sh -e {0}
+      run: |
+        node --version
+        npm --version
+
+    # Get npm cache directory
+    - uses: actions/cache@v2
+      id: npm-cache
+      with:
+        path: ~/.npm
+        key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
+        restore-keys: |
+          ${{ runner.os }}-node-
+
+    # Install node dependencies
+    - shell: sh -e {0}
+      # Update npm cache directory if package-lock has changed
+      run: |
+        if [ "${{ inputs.install-deps }}" = "yes" ] || [ "${{ steps.npm-cache.outputs.cache-hit }}" != "true" ]; then
+          npm ci `if [ "${{ inputs.ignore-scripts }}" = "yes" ]; then printf %s "--ignore-scripts"; fi`
+        else
+          echo "Skip npm ci"
+        fi

.github/workflows/js-css-linting.yml

@@ -15,23 +15,10 @@ jobs:
       - name: Checkout repository
         uses: actions/checkout@v2
 
-      - name: Log debug information
-        run: |
-          node --version
-          npm --version
-
-      - name: Get npm cache directory
-        id: npm-cache
-        uses: actions/cache@v2
+      - name: Prepare node
+        uses: ./.github/actions/prepare-node
         with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-
-
-      - name: Update npm cache directory # if package-lock has changed
-        if: steps.npm-cache.outputs.cache-hit != 'true'
-        run: npm ci --ignore-scripts
+          install-deps: "maybe"
 
   JSLintingCheck:
     name: Lint JavaScript
@@ -41,16 +28,8 @@ jobs:
       - name: Checkout repository
         uses: actions/checkout@v2
 
-      - name: Get npm cache directory
-        uses: actions/cache@v2
-        with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-
-
-      - name: Install Node dependencies
-        run: npm ci --ignore-scripts
+      - name: Prepare node
+        uses: ./.github/actions/prepare-node
 
       - name: Save code linting report JSON
         run: npm run lint:js -- --quiet --output-file eslint_report.json --format json
@@ -77,16 +56,8 @@ jobs:
       - name: Checkout repository
         uses: actions/checkout@v2
 
-      - name: Get npm cache directory
-        uses: actions/cache@v2
-        with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-
-
-      - name: Install Node dependencies
-        run: npm ci --ignore-scripts
+      - name: Prepare node
+        uses: ./.github/actions/prepare-node
 
       - name: Lint CSS
         run: npm run lint:css

.github/workflows/js-unit-tests.yml

@@ -15,21 +15,8 @@ jobs:
       - name: Checkout repository
         uses: actions/checkout@v2
 
-      - name: Log debug information
-        run: |
-          node --version
-          npm --version
-
-      - name: Get npm cache directory
-        uses: actions/cache@v2
-        with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-
-
-      - name: Install Node dependencies
-        run: npm ci --ignore-scripts
+      - name: Prepare node
+        uses: ./.github/actions/prepare-node
 
       - name: Run JavaScript unit tests
         run: npm run test-unit

.github/workflows/php-unit-tests.yml

@@ -15,25 +15,15 @@ jobs:
       - name: Checkout repository
         uses: actions/checkout@v2
 
+      - name: Prepare node
+        uses: ./.github/actions/prepare-node
+        with:
+          install-deps: "maybe"
+
       - name: Log debug information
         run: |
-          node --version
-          npm --version
           composer --version
 
-      - name: Get npm cache directory
-        id: npm-cache
-        uses: actions/cache@v2
-        with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-
-
-      - name: Update npm cache directory # if package-lock has changed
-        if: steps.npm-cache.outputs.cache-hit != 'true'
-        run: npm ci --ignore-scripts
-
       - name: Get Composer cache directory
         id: composer-cache-config
         run: echo "::set-output name=dir::$(composer config cache-files-dir)"
@@ -67,13 +57,8 @@ jobs:
       - name: Checkout repository
         uses: actions/checkout@v2
 
-      - name: Get npm cache directory
-        uses: actions/cache@v2
-        with:
-          path: ~/.npm
-          key: ${{ runner.os }}-node-${{ hashFiles('package-lock.json') }}
-          restore-keys: |
-            ${{ runner.os }}-node-
+      - name: Prepare node
+        uses: ./.github/actions/prepare-node
 
       - name: Get Composer cache directory
         id: composer-cache-config
@@ -104,10 +89,8 @@ jobs:
       - name: Install Composer dependencies
         run: composer install --prefer-dist --no-suggest --no-progress --no-ansi --no-interaction
 
-      - name: Install Node dependencies and build
-        run: |
-          npm ci
-          npm run dev
+      - name: Build client bundles
+        run: npm run dev
 
       - name: Install WP tests
         shell: bash