raducotescu commented on pull request #11: URL: https://github.com/apache/sling-parent/pull/11#issuecomment-701354169
> I think for > > https://github.com/apache/sling-parent/blob/439bb3f192f51e1cbd69d5b3ae5bdb2c7423c92b/pom.xml#L720 > > we use a non-standard location for the full report xml - if we wanted SonarQube to import that I think we have to set `sonar.coverage.jacoco.xmlReportPaths` (see https://community.sonarsource.com/t/coverage-test-data-importing-jacoco-coverage-report-in-xml-format/) - it might be easier to generate the xml just to the default location (then no SonarQube config is needed) That's already handled by https://github.com/apache/sling-tooling-jenkins/commit/bd97a2afd3f32b82985181bd1a7e44f4ca874be3#diff-73dbfe33a23fd613048b067ac78dd258R40. @ghenzler , do you see any value in changing this again? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
