chore: fmt ci uses correct node setup

Emyrk · Emyrk · commit c46e06f3fde9 · 2023-07-05T10:38:35.000-04:00
Add flag for node setup to allow not installing all node_modules
diff --git a/.github/actions/setup-node/action.yaml b/.github/actions/setup-node/action.yaml
@@ -1,6 +1,11 @@
 name: "Setup Node"
 description: |
   Sets up the node environment for tests, builds, etc.
+inputs:
+  yarn-install-flags:
+    description: "Flags to pass to yarn install"
+    required: false
+    default: ""
 runs:
   using: "composite"
   steps:
@@ -12,4 +17,4 @@ runs:
         cache-dependency-path: "site/yarn.lock"
     - name: Install node_modules
       shell: bash
-      run: ./scripts/yarn_install.sh
+      run: ./scripts/yarn_install.sh ${{ inputs.yarn-install-flags }}
diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml
@@ -192,19 +192,18 @@ jobs:
       - name: Checkout
         uses: actions/checkout@v3
 
-      - uses: buildjet/setup-node@v3
+      - uses: ./.github/actions/setup-node
         with:
-          node-version: 16.16.0
+          # Only install prettier dep for this job
+          yarn-install-flags: "prettier"
+
 
       - uses: buildjet/setup-go@v4
         with:
           # This doesn't need caching. It's super fast anyways!
           cache: false
           go-version: 1.20.5
 
-      - name: Install prettier
-        run: npm install -g prettier
-
       - name: Install shfmt
         run: go install mvdan.cc/sh/v3/cmd/shfmt@v3.5.0
 
