Merge PR #44 Patching #30: Always show user list

in slack
jenkinsci · Feb 4, 2015 · ec01e8a · ec01e8a
2 parents 34b0709 + 3ec7d4f
commit ec01e8a
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/main/java/jenkins/plugins/slack/ActiveNotifier.java b/src/main/java/jenkins/plugins/slack/ActiveNotifier.java
@@ -91,10 +91,9 @@ public void completed(AbstractBuild r) {
                 || (result == Result.UNSTABLE && jobProperty.getNotifyUnstable())) {
             getSlack(r).publish(getBuildStatusMessage(r, jobProperty.includeTestSummary()),
                     getBuildColor(r));
-        }
-        if (result == Result.SUCCESS && jobProperty.getNotifySuccess()
-                && jobProperty.getShowCommitList()) {
-            getSlack(r).publish(getCommitList(r), "good");
+            if (jobProperty.getShowCommitList()) {
+                getSlack(r).publish(getCommitList(r), getBuildColor(r));
+            }
         }
     }