From b25088b08c6cad1ca5239ad1e367c764b1e056bd Mon Sep 17 00:00:00 2001 From: Sahil Agarwal Date: Tue, 6 Mar 2018 14:38:53 -0800 Subject: [PATCH 1/2] Add specific pr and branch templates for slack --- lib/travis/addons/slack/task.rb | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) diff --git a/lib/travis/addons/slack/task.rb b/lib/travis/addons/slack/task.rb index 40afe7ef..76d68d6a 100644 --- a/lib/travis/addons/slack/task.rb +++ b/lib/travis/addons/slack/task.rb @@ -61,7 +61,7 @@ def message(channel) end def message_text - lines = Array(template_from_config || default_template) + lines = notification_template lines.map {|line| Util::Template.new(line, payload, source: :slack).interpolate}.join("\n") end @@ -76,21 +76,23 @@ def color end end - def template_from_config - slack_config.is_a?(Hash) ? slack_config[:template] : nil - end - def slack_config build[:config].try(:[], :notifications).try(:[], :slack) || {} end - def default_template - if pull_request? - PULL_REQUEST_MESSAGE_TEMPLATE + def notification_template + if template_from_config(:template) + Array(template_from_config(:template)) + elsif pull_request? + Array(template_from_config(:pr_template) || PULL_REQUEST_MESSAGE_TEMPLATE) else - BRANCH_BUILD_MESSAGE_TEMPLATE + Array(template_from_config(:branch_template) || BRANCH_BUILD_MESSAGE_TEMPLATE) end end + + def template_from_config(key) + slack_config.is_a?(Hash) ? slack_config[key] : nil + end end end end From c694814cf05ba18ecae08745839fe04ca0b54d96 Mon Sep 17 00:00:00 2001 From: Sahil Agarwal Date: Wed, 3 Apr 2019 13:39:54 -0700 Subject: [PATCH 2/2] Rename 'branch_template' to 'branch_build_template' --- lib/travis/addons/slack/task.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/travis/addons/slack/task.rb b/lib/travis/addons/slack/task.rb index 76d68d6a..6a339d92 100644 --- a/lib/travis/addons/slack/task.rb +++ b/lib/travis/addons/slack/task.rb @@ -86,7 +86,7 @@ def notification_template elsif pull_request? Array(template_from_config(:pr_template) || PULL_REQUEST_MESSAGE_TEMPLATE) else - Array(template_from_config(:branch_template) || BRANCH_BUILD_MESSAGE_TEMPLATE) + Array(template_from_config(:branch_build_template) || BRANCH_BUILD_MESSAGE_TEMPLATE) end end