CURATOR-624: Fix Github Actions by adding 'distribution' for JDK setu…

…p step (#407)
apache · Feb 25, 2022 · 6f7b3e7 · 6f7b3e7
1 parent 2884ec4
commit 6f7b3e7
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 54 deletions.
diff --git a/.github/workflows/ci-unit-jdk8.yaml b/.github/workflows/ci-unit-jdk8.yaml
diff --git a/.github/workflows/ci-unit-jdk11.yaml → .github/workflows/ci-unit.yaml b/.github/workflows/ci-unit-jdk11.yaml → .github/workflows/ci-unit.yaml
@@ -17,7 +17,7 @@
 # under the License.
 #
 
-name: CI - Tests JDK 11
+name: CI
 on:
   push:
   pull_request:
@@ -32,18 +32,32 @@ jobs:
     name:
     runs-on: ubuntu-latest
     timeout-minutes: 120
+    strategy:
+      matrix:
+        java:
+        - '8'
+        - '11'
 
     steps:
       - name: Checkout
         uses: actions/checkout@v2
 
-      - name: Set up JDK 11
+      - name: Cache Local Maven Repository
+        uses: actions/cache@v2
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+          restore-keys: |
+            ${{ runner.os }}-maven-
+
+      - name: Set up JDK ${{ matrix.java }}
         uses: actions/setup-java@v2
         with:
-          java-version: 11
+          java-version: ${{ matrix.java }}
+          distribution: 'temurin'
 
-      - name: Build
+      - name: Build with ${{ matrix.java }}
         run: mvn -B clean install -DskipTests
 
-      - name: Test
+      - name: Test with ${{ matrix.java }}
         run: mvn -B verify