Sonarqube no jacoco exclusions

.github/workflows/check-pr.yaml

@@ -18,21 +18,36 @@ jobs:
     runs-on: ${{ matrix.os }}
     steps:
       - uses: actions/checkout@v4
+        with:
+            fetch-depth: 0 #shallow clone for better sonarqube analysis
       - uses: actions/cache@v4
         with:
           path: ~/.fhir
           key: ${{ runner.os }}-fhir-${{ hashFiles('**/*.*') }}
           restore-keys: |
             ${{ runner.os }}-fhir-
+      - uses: actions/cache@v4
+        if: matrix.os == 'ubuntu-latest'
+        with:
+            path: ~/.sonar/cache
+            key: ${{ runner.os }}-sonar
+            restore-keys: ${{ runner.os }}-sonar
       - uses: actions/setup-java@v4
         with:
             distribution: 'temurin'
             java-version: '21'
             cache: 'maven'
       - name: Compile
         run: ./mvnw --batch-mode --no-transfer-progress --update-snapshots -T 4 package -DskipTests=true
-      - name: Tests and additional checks
+      - name: Run tests
         run: ./mvnw --batch-mode --no-transfer-progress -T 4 verify -Ppackage
+      - name: SonarCloud Analysis
+        if: matrix.os == 'ubuntu-latest'
+        env:
+            GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+            SONAR_TOKEN: ${{ secrets.SONAR_TOKEN_L }}
+        run: ./mvnw --batch-mode --no-transfer-progress sonar:sonar -Dsonar.organization=cqframework -Dsonar.projectKey=cqframework_clinical-reasoning -Dsonar.token=$SONAR_TOKEN
+        #        run: ./mvnw --batch-mode --no-transfer-progress org.sonarsource.scanner.maven:sonar-maven-plugin:sonar -Dsonar.organization=cqframework -Dsonar.projectKey=cqframework_clinical-reasoning -Dsonar.token=$SONAR_TOKEN
       - name: Publish test report
         uses: mikepenz/action-junit-report@v4
         if: (success() || failure()) && matrix.os == 'ubuntu-latest'

README.MD

@@ -8,6 +8,11 @@ This project provides a Java-based, FHIR-enabled clinical reasoning components.
 
 See the [docs](https://www.cqframework.org/clinical-reasoning/) for more information.
 
+## SonarCloud
+
+[![Coverage](https://sonarcloud.io/api/project_badges/measure?project=cqframework_clinical-reasoning&metric=coverage)]
+(https://sonarcloud.io/dashboard?id=cqframework_clinical-reasoning)
+
 ## Getting Help
 
 Bugs and feature requests can be filed with [Github Issues](https://github.com/cqframework/clinical-reasoning/issues).

cqf-fhir-benchmark/pom.xml

@@ -15,6 +15,11 @@
         <version>4.3.0-SNAPSHOT</version>
     </parent>
 
+    <!-- this project contains one file with no content - so let's skip checking it -->
+    <properties>
+        <jacoco.skip>true</jacoco.skip>
+    </properties>
+
     <dependencies>
         <dependency>
             <groupId>org.opencds.cqf.fhir</groupId>

cqf-fhir-cr/src/main/java/org/opencds/cqf/fhir/cr/measure/common/StratumPopulationDef.java

@@ -10,7 +10,7 @@
 
 /**
  * Equivalent to the FHIR stratum population.
- * <p/>
+ *
  * This is meant to be the source of truth for all data points regarding stratum populations.
  */
 public record StratumPopulationDef(

pom.xml

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <project xmlns="http://maven.apache.org/POM/4.0.0"
-    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-    xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
     <modelVersion>4.0.0</modelVersion>
 
     <groupId>org.opencds.cqf.fhir</groupId>
@@ -30,6 +30,22 @@
         <guava.version>33.2.1-jre</guava.version>
         <error-prone.version>2.35.1</error-prone.version>
         <checkstyle.version>10.12.7</checkstyle.version>
+
+        <!-- SonarCloud configuration -->
+        <sonar.organization>cqframework</sonar.organization>
+        <sonar.projectKey>cqframework_clinical-reasoning</sonar.projectKey>
+        <sonar.host.url>https://sonarcloud.io</sonar.host.url>
+
+        <!-- Coverage settings for jacoco -->
+        <sonar.coverage.jacoco.xmlReportPaths>${project.build.directory}/site/jacoco/jacoco.xml</sonar.coverage.jacoco.xmlReportPaths>
+        <sonar.java.coveragePlugin>jacoco</sonar.java.coveragePlugin>
+
+        <!-- Exclusions -->
+        <sonar.exclusions>**/generated/**,**/benchmark/**</sonar.exclusions>
+        <sonar.coverage.exclusions>**/config/**,**/constants/**</sonar.coverage.exclusions>
+
+        <!-- Multi-module aggregation -->
+        <sonar.moduleKey>${project.groupId}:${project.artifactId}</sonar.moduleKey>
     </properties>
 
     <licenses>
@@ -55,9 +71,9 @@
     </developers>
 
     <repositories>
-         <repository>
+        <repository>
             <id>central</id>
-                <url>https://repo1.maven.org/maven2</url>
+            <url>https://repo1.maven.org/maven2</url>
             <releases>
                 <enabled>true</enabled>
             </releases>
@@ -273,7 +289,7 @@
         <dependency>
             <groupId>jakarta.annotation</groupId>
             <artifactId>jakarta.annotation-api</artifactId>
-      <scope>provided</scope>
+            <scope>provided</scope>
         </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>
@@ -574,6 +590,10 @@
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-failsafe-plugin</artifactId>
             </plugin>
+            <plugin>
+                <groupId>org.jacoco</groupId>
+                <artifactId>jacoco-maven-plugin</artifactId>
+            </plugin>
         </plugins>
     </build>
     <profiles>