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

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


The following commit(s) were added to refs/heads/feature/SLING-7245 by this 
push:
     new 5560191  SLING-7245 - Validate pull requests using Jenkins
5560191 is described below

commit 5560191e958f9d2111b6dbc50ef1ff8bfce35be1
Author: Robert Munteanu <[email protected]>
AuthorDate: Thu Dec 20 16:46:05 2018 +0100

    SLING-7245 - Validate pull requests using Jenkins
    
    Wrap notifications in a stage, also use defined recipients.
---
 vars/slingOsgiBundleBuild.groovy | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/vars/slingOsgiBundleBuild.groovy b/vars/slingOsgiBundleBuild.groovy
index 8942b54..7386994 100644
--- a/vars/slingOsgiBundleBuild.groovy
+++ b/vars/slingOsgiBundleBuild.groovy
@@ -101,7 +101,9 @@ def call(Map params = [:]) {
             currentBuild.result = "FAILURE"
             throw e
         } finally {
-            processResult(currentBuild, 
currentBuild.getPreviousBuild()?.result, jobConfig['emailRecipients'])
+            stage("Notifications") {
+                processResult(currentBuild, 
currentBuild.getPreviousBuild()?.result, jobConfig['emailRecipients'])
+            }
         }
     }
 }
@@ -149,5 +151,5 @@ def processResult(def currentBuild, String previous, def 
recipients) {
         ${BUILD_LOG}
     """
 
-    emailExt subject: subject, body: body, replyTo: '[email protected]', 
recipientProviders: recipientProviders, to: '[email protected]'
+    emailExt subject: subject, body: body, replyTo: '[email protected]', 
recipientProviders: recipientProviders, to: recipients.join(',')
 }
\ No newline at end of file

Reply via email to