Run ninja build before tests (#5470)

* Run ninja build before tests

* Separate ninja stuff into multiple steps

Author: cknitt

.github/workflows/ci.yml

@@ -57,6 +57,17 @@ jobs:
         env:
           RESCRIPT_FORCE_REBUILD: 1
 
+      - name: ninja config
+        if: runner.os != 'Windows'
+        run: opam exec -- node ./scripts/ninja.js config
+
+      - name: ninja build
+        if: runner.os != 'Windows'
+        run: opam exec -- node ./scripts/ninja.js build
+
+      - name: Check for changes in lib folder
+        run: git diff --exit-code lib/js lib/es6
+
       - name: Run tests
         if: runner.os != 'Windows'
         run: opam exec -- node scripts/ciTest.js -all
@@ -65,13 +76,6 @@ jobs:
         if: runner.os == 'Windows'
         run: opam exec -- node scripts/ciTest.js -mocha -theme
 
-      - name: Ensure lib/js and lib/es6 are up to date
-        if: runner.os != 'Windows'
-        run: |
-          opam exec -- node ./scripts/ninja.js config
-          opam exec -- node ./scripts/ninja.js build
-          git diff --exit-code lib/js lib/es6
-
       - name: Get artifact info
         id: get_artifact_info
         run: node .github/workflows/get_artifact_info.js