Sonar build improvement

PicnicSupermarket · Apr 28, 2023 · aa78360 · aa78360
1 parent fda1f3f
commit aa78360
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/.github/workflows/sonarcloud.yml b/.github/workflows/sonarcloud.yml
@@ -33,4 +33,4 @@ jobs:
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
-        run: mvn -T1C jacoco:prepare-agent verify jacoco:report sonar:sonar -Dversion.jdk.runtime=17 -Dverification.skip -Dsonar.projectKey=PicnicSupermarket_error-prone-support
+        run: mvn -T1C jacoco:prepare-agent verify jacoco:report sonar:sonar -Dverification.skip -Dsonar.projectKey=PicnicSupermarket_error-prone-support
diff --git a/pom.xml b/pom.xml
@@ -1858,6 +1858,12 @@
             <id>sonar</id>
             <properties>
                 <maven.test.failure.ignore>true</maven.test.failure.ignore>
+                <!-- Normal builds use Jabel to target an older JDK version
+                than would normally supported given the language features used,
+                but this causes Sonar to report parse errors. So for Sonar
+                builds we target a JDK version that is properly compatible with
+                the source code.-->
+                <version.jdk.runtime>${version.jdk.source}</version.jdk.runtime>
             </properties>
         </profile>
         <profile>