You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by js...@apache.org on 2020/02/28 16:13:12 UTC

[sling-tooling-jenkins] 01/01: SLING-8995 - Code coverage no longer picked up by SonarCloud

This is an automated email from the ASF dual-hosted git repository.

jsedding pushed a commit to branch SLING-8995
in repository https://gitbox.apache.org/repos/asf/sling-tooling-jenkins.git

commit 383e7bc011e738ee34f5c3269ff01cba3e503998
Author: Julian Sedding <js...@apache.org>
AuthorDate: Fri Feb 28 17:10:40 2020 +0100

    SLING-8995 - Code coverage no longer picked up by SonarCloud
    
    - change to sonar.coverage.jacoco.xmlReportPaths property, sonar.jacoco.reportPaths is deprecated
---
 vars/slingOsgiBundleBuild.groovy | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/vars/slingOsgiBundleBuild.groovy b/vars/slingOsgiBundleBuild.groovy
index 5ee02c9..9e6b884 100644
--- a/vars/slingOsgiBundleBuild.groovy
+++ b/vars/slingOsgiBundleBuild.groovy
@@ -36,7 +36,7 @@ def call(Map params = [:]) {
 
                 stage('SonarCloud') {
                     // As we don't have the global SonarCloud conf for now, we can't use #withSonarQubeEnv so we need to set the following props manually
-                    def sonarcloudParams="-Dsonar.host.url=https://sonarcloud.io -Dsonar.organization=apache -Dsonar.projectKey=apache_${jobConfig.repoName} -Pjacoco-report -Dsonar.jacoco.reportPaths=target/jacoco-merged.exec ${additionalMavenParams}"
+                    def sonarcloudParams="-Dsonar.host.url=https://sonarcloud.io -Dsonar.organization=apache -Dsonar.projectKey=apache_${jobConfig.repoName} -Pjacoco-report -Dsonar.coverage.jacoco.xmlReportPaths=target/site/jacoco-merged/jacoco.xml ${additionalMavenParams}"
                     // Params are different if it's a PR or if it's not
                     // Note: soon we won't have to handle that manually, see https://jira.sonarsource.com/browse/SONAR-11853
                     if ( isPrBuild ) {