CI: Drop JDK 11 form the matrix; employ maven-wrapper for determinist…

…ic builds.
rhusar · Dec 17, 2024 · 233dc06 · 233dc06
1 parent 54b3d31
commit 233dc06
Showing 1 changed file with 13 additions and 28 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -12,52 +12,37 @@ jobs:
     strategy:
       matrix:
         os: [ ubuntu-latest, windows-latest, macos-latest ]
-        # Keep this list as: all supported >= 17 LTS JDKs
-        java-compilation: [ 17, 21 ]
         # Keep this list as: all supported LTS JDKs, the latest GA JDK, and optionally the latest EA JDK (if available).
         # https://www.oracle.com/java/technologies/java-se-support-roadmap.html
-        java: [ 11, 17, 21, 23 ]
+        java: [ 17, 21, 23 ]
     steps:
       - name: Checkout
         uses: actions/checkout@v4
-      - name: Set up JDK ${{ matrix.java-compilation }} (compilation)
-        uses: actions/setup-java@v4
-        with:
-          distribution: temurin
-          java-version: ${{ matrix.java-compilation }}
-      - name: Build with Maven using JDK ${{ matrix.java-compilation }} (compilation only)
-        run: mvn --batch-mode --no-transfer-progress verify --define skipTests=true
-      - name: Cache local Maven repository
-        uses: actions/cache@v4
-        with:
-          path: ~/.m2/repository
-          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
-          restore-keys: |
-            ${{ runner.os }}-maven-
-      - name: Set up JDK ${{ matrix.java }} (runtime)
+      - name: Set up JDK ${{ matrix.java }}
         uses: actions/setup-java@v4
         with:
           distribution: temurin
           java-version: ${{ matrix.java }}
-      - name: Build with Maven using JDK ${{ matrix.java }}
+          cache: maven
+      - name: Build with Maven
         # To support Windows PowerShell, a space between --define property=true is required.
-        run: mvn --batch-mode --no-transfer-progress --define maven.main.skip=true verify
+        run: ./mvnw --batch-mode --no-transfer-progress verify
       # Code Coverage (runs once per matrix)
-      - name: Build with coverage with Maven using JDK ${{ matrix.runtime }}
-        if: ${{ matrix.java-compilation == '17' && matrix.java == '11' && matrix.os == 'ubuntu-latest' }}
-        run: mvn --batch-mode --no-transfer-progress --activate-profiles=coverage --define maven.main.skip=true verify
+      - name: Build with coverage with Maven using JDK ${{ matrix.java }}
+        if: ${{ matrix.java == '17' && matrix.os == 'ubuntu-latest' }}
+        run: ./mvnw --batch-mode --no-transfer-progress --activate-profiles=coverage verify
       - name: Upload JaCoCo report
-        if: ${{ matrix.java-compilation == '17' && matrix.java == '11' && matrix.os == 'ubuntu-latest' }}
+        if: ${{ matrix.java == '17' && matrix.os == 'ubuntu-latest' }}
         uses: actions/upload-artifact@v4
         with:
           name: jacoco
           path: code-coverage/target/site/jacoco-aggregate/
       # Aggregate Javadoc (runs once per matrix)
-      - name: Build aggregate Javadoc with Maven using JDK ${{ matrix.runtime }}
-        if: ${{ matrix.java-compilation == '17' && matrix.java == '17' && matrix.os == 'ubuntu-latest' }}
-        run: mvn --batch-mode --no-transfer-progress --define skipTests=true verify javadoc:aggregate
+      - name: Build aggregate Javadoc with Maven using JDK ${{ matrix.java }}
+        if: ${{ matrix.java == '17' && matrix.os == 'ubuntu-latest' }}
+        run: ./mvnw --batch-mode --no-transfer-progress --define skipTests=true verify javadoc:aggregate
       - name: Upload aggregate Javadoc
-        if: ${{ matrix.java-compilation == '17' && matrix.java == '17' && matrix.os == 'ubuntu-latest' }}
+        if: ${{ matrix.java == '17' && matrix.os == 'ubuntu-latest' }}
         uses: actions/upload-artifact@v4
         with:
           name: apidocs