fix(ci): fine tune steps

HistoryAtState · Mar 23, 2024 · 28063d9 · 28063d9
1 parent 93378eb
commit 28063d9
Showing 1 changed file with 34 additions and 23 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -8,10 +8,12 @@ name: compile and test
 on: [push, pull_request]
 
 jobs:
-  static:
+  build:
     runs-on: ubuntu-latest
     strategy:
-       fail-fast: true
+       matrix:
+         java-version: [11, 17]
+
 
     steps:
       # Checkout code
@@ -38,26 +40,35 @@ jobs:
           xmllint --noout \
             $(find . -type f -name '*.xml')
 
+      - name: Build Expath Package
+        uses: actions/setup-java@v4
+        with:
+          distribution: 'temurin'
+          java-version: ${{ matrix.java-version }}
+      - run: ant -Dapp.version=1.0.0-SNAPSHOT
+
+      - name: upload xar package
+        if: ${{ matrix.java-version == 17 }}
+        uses: actions/upload-artifact@v4
+        with:
+          name: frus-xar
+          path: build/*.xar
+          if-no-files-found: ignore
+
       # Commit Toc files
       - name: Commit and Push
-        if: ${{ matrix.java-version == 17}}
+        if: ${{ matrix.java-version == 17 }}
         uses: actions-x/commit@v6
         with:
           message: "fix(toc): update [skip ci]"
-  build:
+  test:
     runs-on: ubuntu-latest
-    needs: static
+    needs: build
     strategy:
        fail-fast: false
        matrix:
         # 7.0.0-SNAPSHOT and 6.2.1 created
          exist-version: [latest, release]
-         java-version: [11, 17]
-         exclude:
-           - exist-version: release
-             java-version: 17
-           - exist-version: latest
-             java-version: 11
 
     steps:
       - name: Maximize build space
@@ -74,14 +85,22 @@ jobs:
         run: sudo service docker restart
 
       - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v3  
+        uses: docker/setup-buildx-action@v3 
 
-      # Checkout code
-      - uses: actions/checkout@v4
       - name: Install Test Dependencies
         run: |
           sudo apt-get update
           sudo apt-get install -y bats
+      
+      - name: create build/
+        run: mkdir build
+
+      # TODO avoid 2n checkout and re-use move expath to previous step
+      # mind java matrix for build?
+      - uses: actions/download-artifact@v4
+        with:
+          name: frus-xar
+          path: build
 
     #  sanity check
       - name: check os
@@ -95,14 +114,6 @@ jobs:
       - name: Check config
         run: docker info
 
-      # Build 
-      - name: Build Expath Package
-        uses: actions/setup-java@v4
-        with:
-          distribution: 'temurin'
-          java-version: ${{ matrix.java-version }}
-      - run: ant -Dapp.version=1.0.0-SNAPSHOT
-
       # Install
       - name: Start exist-ci containers
         run: |
@@ -142,7 +153,7 @@ jobs:
   release:
     name: Release
     runs-on: ubuntu-latest
-    needs: build
+    needs: test
     if: github.ref == 'refs/heads/master'
     steps:
       - name: Checkout