Skip to content

OAK-12140: use aggregated coverage report in Sonar checks#2802

Open
reschke wants to merge 2 commits intotrunkfrom
OAK-12140
Open

OAK-12140: use aggregated coverage report in Sonar checks#2802
reschke wants to merge 2 commits intotrunkfrom
OAK-12140

Conversation

@reschke
Copy link
Contributor

@reschke reschke commented Mar 18, 2026

No description provided.

@reschke reschke marked this pull request as draft March 18, 2026 14:04
@reschke reschke requested a review from kwin March 18, 2026 19:05
@reschke reschke marked this pull request as ready for review March 18, 2026 19:05
@reschke
Copy link
Contributor Author

reschke commented Mar 18, 2026

I believe this is correct, but how do I test?

@reschke reschke requested a review from kwin March 19, 2026 06:01
@sonarqubecloud
Copy link

${project.reporting.outputDirectory}/jacoco-ut/jacoco.xml
</sonar.coverage.jacoco.xmlReportPaths>
<!-- this project uses cross-module reports with one aggregate, https://docs.sonarsource.com/sonarcloud/enriching/test-coverage/java-test-coverage/#add-coverage-in-a-multi-module-maven-project -->
<!-- "maven.multiModuleProjectDirectory" is undocumented (https://issues.apache.org/jira/browse/MNG-6589) but the only way to reference the aggregate root directory in Maven 3.x-->
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://issues.apache.org/jira/browse/MNG-7038 (which is linked from the closed issues) talks about a possible alternative? I'm not sure...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see epic discussions but now solutions. @kwin was involved there, so he would know.

For now this is exactly what we do in Jackrabbit (classic).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MNG-7038 is about a new solution in the upcoming Maven4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants