Run Maven samples w/ both 25 and 17

This commit updates both ci-pr.yml and deploy.yml to build and
test w/ Java 25 and run Maven samples w/ Java 17.

See #290

Signed-off-by: onobc <[email protected]>

Author: onobc

.github/workflows/ci-pr.yml

@@ -15,23 +15,30 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - name: Set up JDK 17 (0.x)
-        if: github.base_ref == '0.x'
-        uses: actions/setup-java@v4
-        with:
-          java-version: '17'
-          distribution: 'temurin'
-          cache: maven
-      - name: Set up JDK 25 (main)
-        if: github.base_ref == 'main'
+      - name: Set up JDK 25
         uses: actions/setup-java@v4
         with:
           java-version: '25'
           distribution: 'liberica'
           cache: maven
-      - name: Build and test
+      - name: Build, test, and run samples (Java 25)
         run: |
           ./mvnw -B clean install
+
+      - name: Set up JDK 17
+        uses: actions/setup-java@v4
+        with:
+          java-version: '17'
+          distribution: 'temurin'
+          cache: maven
+      - name: Run samples w/ Java 17
+        run: |
+          pushd samples > /dev/null
+          ../mvnw clean install \
+              -Dmaven-checkstyle-plugin.config.location=../src/checkstyle \
+              -Djava.version=17
+          popd > /dev/null
+
       - name: Capture Test Results
         if: failure()
         uses: actions/upload-artifact@v4

.github/workflows/deploy.yml

@@ -3,7 +3,7 @@
 name: Deploy Jars
 on:
   push:
-    branches: [ main, 0.x ]
+    branches: [ main ]
 
 jobs:
   build:
@@ -14,15 +14,31 @@ jobs:
 
     steps:
     - uses: actions/checkout@v4
-    - name: Set up JDK 17 (0.x)
-      if: github.ref != 'refs/heads/main'
+    - name: Set up JDK 25
+      uses: actions/setup-java@v4
+      with:
+        java-version: '25'
+        distribution: 'liberica'
+        cache: maven
+    - name: Build, test, and run samples (Java 25)
+      run: |
+        ./mvnw -B clean install
+
+    - name: Set up JDK 17
       uses: actions/setup-java@v4
       with:
         java-version: '17'
         distribution: 'temurin'
         cache: maven
-    - name: Set up JDK 25 (main)
-      if: github.ref == 'refs/heads/main'
+    - name: Run samples w/ Java 17
+      run: |
+        pushd samples > /dev/null
+        ../mvnw clean install \
+            -Dmaven-checkstyle-plugin.config.location=../src/checkstyle \
+            -Djava.version=17
+        popd > /dev/null
+
+    - name: Set up JDK 25
       uses: actions/setup-java@v4
       with:
         java-version: '25'

pom.xml

@@ -118,6 +118,7 @@
 		<spring-javaformat-checkstyle.version>0.0.43</spring-javaformat-checkstyle.version>
 		<disable.format.checks>false</disable.format.checks>
 		<disable.checkstyle.checks>false</disable.checkstyle.checks>
+		<maven-checkstyle-plugin.config.location>src/checkstyle</maven-checkstyle-plugin.config.location>
 		<maven-checkstyle-plugin.failsOnError>true</maven-checkstyle-plugin.failsOnError>
 		<maven-checkstyle-plugin.failOnViolation>true</maven-checkstyle-plugin.failOnViolation>
 		<maven-checkstyle-plugin.includeTestSourceDirectory>true</maven-checkstyle-plugin.includeTestSourceDirectory>
@@ -165,9 +166,9 @@
 						<phase>validate</phase>
 						<inherited>true</inherited>
 						<configuration>
-							<configLocation>src/checkstyle/checkstyle.xml</configLocation>
+							<configLocation>${maven-checkstyle-plugin.config.location}/checkstyle.xml</configLocation>
 							<propertyExpansion>
-								config_loc=src/checkstyle
+								config_loc=${maven-checkstyle-plugin.config.location}
 							</propertyExpansion>
 							<skip>${disable.checkstyle.checks}</skip>
 							<includeTestSourceDirectory>${maven-checkstyle-plugin.includeTestSourceDirectory}</includeTestSourceDirectory>