Fix uberjar build

- Build the frontend first - Runs the build as a separate job from tests
nilenso · Sep 3, 2020 · e1d0b53 · e1d0b53
1 parent e90285a
commit e1d0b53
Showing 1 changed file with 34 additions and 12 deletions.
diff --git a/.github/workflows/chronograph.yml b/.github/workflows/chronograph.yml
@@ -16,7 +16,7 @@ env:
   CHRONOGRAPH_JAR: chronograph.jar
 
 jobs:
-  server-build:
+  server-test:
     services:
       postgres:
         image: postgres:12.3
@@ -54,16 +54,7 @@ jobs:
     - name: Run tests
       run: lein test
 
-    - name: Build uberjar
-      run: lein uberjar
-
-    - name: Upload uberjar
-      uses: actions/upload-artifact@v2
-      with:
-        name: chronograph-uberjar
-        path: target/${{env.CHRONOGRAPH_JAR}}
-
-  frontend-build:
+  frontend-test:
     runs-on: ubuntu-latest
 
     steps:
@@ -75,10 +66,41 @@ jobs:
     - name: Run frontend tests
       run: yarn test
 
+  uberjar-build:
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v2
+
+    - name: Cache lein deps
+      uses: actions/cache@v2
+      env:
+        cache-name: cache-lein-deps
+      with:
+        path: ~/.m2
+        key: ${{ env.cache-name }}-${{ hashFiles('**/project.clj') }}
+        restore-keys: |
+          ${{ env.cache-name }}-
+
+    - name: Install frontend deps
+      run: yarn install
+
+    - name: Build the frontend
+      run: yarn release
+
+    - name: Build uberjar
+      run: lein uberjar
+
+    - name: Upload uberjar
+      uses: actions/upload-artifact@v2
+      with:
+        name: chronograph-uberjar
+        path: target/${{env.CHRONOGRAPH_JAR}}
+
   staging-deploy:
     runs-on: ubuntu-latest
 
-    needs: server-build
+    needs: uberjar-build
 
     env:
       CHRONOGRAPH_STAGING_INVENTORY: "staging.hosts"