git-commit-notifier 0.11.2 → 0.11.3

data/VERSION

@@ -1 +1 @@
-0.11.2
+0.11.3

data/git-commit-notifier.gemspec

@@ -36,7 +36,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency(%q<rspec-expectations>, [">= 0"])
   s.add_development_dependency(%q<rr>, ["~> 1.0"])
   s.add_development_dependency(%q<faker>, ["~> 0.9.5"])
-  s.add_development_dependency(%q<yard>, ["~> 0.7.4"])
+  s.add_development_dependency(%q<yard>, ["~> 0.7.5"])
   s.add_development_dependency(%q<redcarpet>, ["~> 1.17.2"])
 end
 

data/lib/git_commit_notifier/commit_hook.rb

@@ -6,28 +6,43 @@ require 'net/smtp'
 require 'digest/sha1'
 
 module GitCommitNotifier
+  # Represents Git commit hook handler.
   class CommitHook
 
     class << self
-      attr_reader :config    
+      # Configuration that read from YAML file.
+      # @return [Hash] Configuration.
+      attr_reader :config
 
+      # Prints error message to $stderr.
+      # @param [String] message Message to be printed to $stderr.
+      # @return [NilClass] nil
       def show_error(message)
         $stderr.puts "************** GIT NOTIFIER PROBLEM *******************"
         $stderr.puts "\n"
         $stderr.puts message
         $stderr.puts "\n"
         $stderr.puts "************** GIT NOTIFIER PROBLEM *******************"
+        nil
       end
 
+      # Prints informational message to $stdout.
+      # @param [String] message Message to be printed to $stdout.
+      # @return [NilClass] nil
       def info(message)
         $stdout.puts message
         $stdout.flush
+        nil
       end
 
+      # Gets logger.
       def logger
         @logger ||= Logger.new(config)
       end
 
+      # Gets list of branches from {config} to include into notifications.
+      # @note All branches will be notified about if returned list is nil; otherwise only specified branches will be notifified about.
+      # @return [Array(String), NilClass] Array of branches to include into notifications or nil.
       def include_branches
         include_branches = config["include_branches"]
         unless include_branches.nil?
@@ -39,19 +54,28 @@ module GitCommitNotifier
         include_branches
       end
 
-      def merge_commit?(result)
-        ! result[:commit_info][:merge].nil?
+      # Is merge commit?
+      # @param [Hash] commit_info Information about commit.
+      def merge_commit?(commit_info)
+        ! commit_info[:commit_info][:merge].nil?
       end
 
+      # Runs comit hook handler using specified arguments.
+      # @param [String] config_name Path to the application configuration file in YAML format.
+      # @param [String] rev1 First specified revision.
+      # @param [String] rev2 Second specified revision.
+      # @param [String] ref_name Git reference (usually in "refs/heads/branch" format).
+      # @return [NilClass] nil
+      # @see config
       def run(config_name, rev1, rev2, ref_name)
-      
-      	# Load the configuration
+
+        # Load the configuration
         @config = File.exists?(config_name) ? YAML::load_file(config_name) : {}
 
         project_path = Git.git_dir
         repo_name = Git.repo_name
         prefix = config["emailprefix"] || repo_name
-        
+
         branch_name = if ref_name =~ /^refs\/heads\/(.+)$/
           $1
         else
@@ -60,7 +84,7 @@ module GitCommitNotifier
         slash_branch_name = "/#{branch_name}"
         slash_branch_name = "" if !config["show_master_branch_name"] && slash_branch_name == '/master'
 
-		# Identify email recipients
+        # Identify email recipients
         recipient = config["mailinglist"] || Git.mailing_list_address
 
         # If no recipients specified, bail out gracefully. This is not an error, and might be intentional
@@ -69,7 +93,7 @@ module GitCommitNotifier
           return
         end
 
-		# Debug information
+        # Debug information
         logger.debug('----')
         logger.debug("cwd: #{Dir.pwd}")
         logger.debug("Git Directory: #{project_path}")
@@ -84,9 +108,9 @@ module GitCommitNotifier
 
         unless include_branches.nil? || include_branches.include?(branch_name)
           info("Supressing mail for branch #{branch_name}...")
-          return
+          return nil
         end
-        
+
         # Replacements for subject template
         #     prefix
         #     repo_name
@@ -110,7 +134,7 @@ module GitCommitNotifier
           :commit_count_phrase => nil,
           :commit_count_phrase2 => nil
         }
-        
+
         info("Sending mail...")
 
         diff2html = DiffToHtml.new(config)
@@ -133,7 +157,7 @@ module GitCommitNotifier
             text << result[:text_content]
             html << result[:html_content]
           end
-          
+
           # Form the subject from template
           revised_subject_words = subject_words.merge({
             :commit_id => result[:commit_info][:commit],
@@ -152,6 +176,7 @@ module GitCommitNotifier
             :from_address => config["from"] || result[:commit_info][:email],
             :from_alias => result[:commit_info][:author],
             :subject => subject,
+            :date => result[:commit_info][:date],
             :text_message => text.join("------------------------------------------\n\n"),
             :html_message => html.join("<hr /><br />"),
             :old_rev => rev1,
@@ -163,8 +188,8 @@ module GitCommitNotifier
         else
           commit_number = 1
           diff2html.diff_between_revisions(rev1, rev2, prefix, ref_name) do |result|
-            next if config["ignore_merge"] && merge_commit?(result)
-            
+            next  if config["ignore_merge"] && merge_commit?(result)
+
             # Form the subject from template
             revised_subject_words = subject_words.merge({
               :commit_id => result[:commit_info][:commit],
@@ -176,13 +201,14 @@ module GitCommitNotifier
             })
             subject_template = config['subject'] || "[${prefix}${slash_branch_name}][${commit_number}] ${message}"
             subject = subject_template.gsub(/\$\{(\w+)\}/) { |m| revised_subject_words[$1.intern] }
-            
+
             emailer = Emailer.new(config,
               :project_path => project_path,
               :recipient => recipient,
               :from_address => config["from"] || result[:commit_info][:email],
               :from_alias => result[:commit_info][:author],
               :subject => subject,
+              :date => result[:commit_info][:date],
               :text_message => result[:text_content],
               :html_message => result[:html_content],
               :old_rev => rev1,
@@ -194,11 +220,16 @@ module GitCommitNotifier
             commit_number += 1
           end
         end
+        nil
       end
 
+      # Gets human readable commit number.
+      # @param [Fixnum] i Commit index.
+      # @return [String] Human readable commit number.
       def number(i)
         "[#{i + 1}]"
       end
     end
   end
 end
+

data/lib/git_commit_notifier/diff_callback.rb

@@ -1,24 +1,30 @@
 # -*- coding: utf-8; mode: ruby; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- vim:fenc=utf-8:filetype=ruby:et:sw=2:ts=2:sts=2
 
 module GitCommitNotifier
+  # Callback for Diff::LCS.traverse_balanced method.
   class DiffCallback
+    # Gets collected tags.
+    # @return [Array(Hash)] Collected tags.
     attr_reader :tags
 
     def initialize
       @tags = []
     end
 
+    # Adds diff match to {#tags}.
     def match(event)
       @tags << { :action => :match, :token => event.old_element }
     end
 
+    # Adds discarded B side to {#tags}.
     def discard_b(event)
       @tags << { :action => :discard_b, :token => event.new_element }
     end
 
+    # Adds discarded A side to {#tags}.
     def discard_a(event)
       @tags << { :action => :discard_a, :token => event.old_element }
     end
-
   end
 end
+

data/lib/git_commit_notifier/diff_to_html.rb

@@ -7,9 +7,11 @@ require 'time'
 require 'git_commit_notifier/escape_helper'
 
 module GitCommitNotifier
+  # Translates Git diff to HTML format
   class DiffToHtml
     include EscapeHelper
 
+    # Integration map for commit message keywords to third-party links.
     INTEGRATION_MAP = {
       :mediawiki => { :search_for => /\[\[([^\[\]]+)\]\]/, :replace_with => '#{url}/\1' },
       :redmine => {
@@ -32,11 +34,9 @@ module GitCommitNotifier
       :bugzilla => { :search_for => /\bBUG\s*(\d+)/i, :replace_with => '#{url}/show_bug.cgi?id=\1' },
       :fogbugz => { :search_for => /\bbugzid:\s*(\d+)/i, :replace_with => '#{url}\1' }
     }.freeze
+    # Maximum email line length in characters.
     MAX_LINE_LENGTH = 512
-    MAX_COMMITS_PER_ACTION = 10000
-    HANDLED_COMMITS_FILE = 'previously.txt'.freeze
-    NEW_HANDLED_COMMITS_FILE = 'previously_new.txt'.freeze
-    GIT_CONFIG_FILE = File.join('.git', 'config').freeze
+    # Number of seconds per day.
     SECS_PER_DAY = 24 * 60 * 60
 
     attr_accessor :file_prefix, :current_file_name
@@ -55,11 +55,13 @@ module GitCommitNotifier
       matches[1..2].map { |m| m.split(',')[0].to_i }
     end
 
+    # Gets HTML class for specified diff line data.
+    # @param [Hash] line Diff line data
     def line_class(line)
       case line[:op]
-      when :removal  then ' class="r"'
-      when :addition then ' class="a"'
-      else                ''
+      when :removal;  ' class="r"'
+      when :addition; ' class="a"'
+      else            ''
       end
     end
 
@@ -70,20 +72,29 @@ module GitCommitNotifier
       end
     end
 
+    # Gets lines_per_diff setting from {#config}.
     def lines_per_diff
       @config['lines_per_diff']
     end
 
+    # Gets ignore_whitespace setting from {#config}.
+    # @return [Boolean] true if whitespaces should be ignored in diff; otherwise false.
     def ignore_whitespaces?
       @config['ignore_whitespace'].nil? || @config['ignore_whitespace']
     end
 
+    # Adds separator between diff blocks to @diff_result.
+    # @return [NilClass] nil
     def add_separator
       @diff_result << '<tr class="sep"><td class="sep" colspan="3" title="Unchanged content skipped between diff. blocks">&hellip;</td></tr>'
+      nil
     end
 
+    # Adds notification to @diff_result about skipping of diff tail due to its large size.
+    # @return [NilClass] nil
     def add_skip_notification
       @diff_result << '<tr><td colspan="3">Diff too large and stripped&hellip;</td></tr>'
+      nil
     end
 
     def add_line_to_result(line, escape)
@@ -127,6 +138,9 @@ module GitCommitNotifier
       result
     end
 
+    # Gets array of tokens from specified str.
+    # @param [String] str Text to be splitted into tokens.
+    # @return [Array(String)] Array of tokens.
     def tokenize_string(str)
       # tokenize by non-word characters
       tokens = []
@@ -157,11 +171,11 @@ module GitCommitNotifier
       end
 
       file_name = @current_file_name
-      
+
       # TODO: these filenames, etc, should likely be properly html escaped
       if config['link_files']
         file_name = if config["link_files"] == "gitweb" && config["gitweb"]
-          "<a href='#{config['gitweb']['path']}?p=#{Git.repo_name}.git;f=#{file_name};h=#{@current_sha};hb=#{@current_commit}'>#{file_name}</a>"
+          "<a href='#{config['gitweb']['path']}?p=#{config['gitweb']['project'] || "#{Git.repo_name}.git"};f=#{file_name};h=#{@current_sha};hb=#{@current_commit}'>#{file_name}</a>"
         elsif config["link_files"] == "gitorious" && config["gitorious"]
           "<a href='#{config['gitorious']['path']}/#{config['gitorious']['project']}/#{config['gitorious']['repository']}/blobs/#{branch_name}/#{file_name}'>#{file_name}</a>"
         elsif config["link_files"] == "cgit" && config["cgit"]
@@ -183,6 +197,8 @@ module GitCommitNotifier
       "<h2>#{header}</h2>\n"
     end
 
+    # Determines are two lines are sequentially placed in diff (no skipped lines between).
+    # @return [Boolean] true if lines are sequential; otherwise false.
     def lines_are_sequential?(first, second)
       result = false
       [:added, :removed].each do |side|
@@ -195,14 +211,14 @@ module GitCommitNotifier
 
     def add_changes_to_result
       return if @current_file_name.nil?
-      
+
       @lines_added = 0
       @diff_result << operation_description
       if !@diff_lines.empty? && !@too_many_files
         @diff_result << '<table>'
         removals = []
         additions = []
-        
+
         lines = if lines_per_diff.nil?
           line_budget = nil
           @diff_lines
@@ -210,7 +226,7 @@ module GitCommitNotifier
           line_budget = lines_per_diff - @lines_added
           @diff_lines.slice(0, line_budget)
         end
-        
+
         lines.each_with_index do |line, index|
           removals << line if line[:op] == :removal
           additions << line if line[:op] == :addition
@@ -230,7 +246,7 @@ module GitCommitNotifier
           end
           @lines_added += 1
         end
-        
+
         add_skip_notification if !line_budget.nil? && line_budget < @diff_lines.size
 
         @diff_result << '</table>'
@@ -336,7 +352,7 @@ module GitCommitNotifier
     def extract_commit_info_from_git_show_output(content)
       result = { :message => [], :commit => '', :author => '', :date => '', :email => '',
       :committer => '', :commit_date => '', :committer_email => ''}
-      
+
       message = []
       content.split("\n").each do |line|
         if line =~ /^diff/ # end of commit info
@@ -357,7 +373,7 @@ module GitCommitNotifier
           message << line.strip
         end
       end
-      
+
       # Strip blank lines off top and bottom of message
       while !message.empty? && message.first.empty?
         message.shift
@@ -366,7 +382,7 @@ module GitCommitNotifier
         message.pop
       end
       result[:message] = message
-      
+
       result
     end
 
@@ -404,7 +420,7 @@ module GitCommitNotifier
     def merge_commit?(commit_info)
       ! commit_info[:merge].nil?
     end
-    
+
     def truncate_long_lines(text)
       StringIO.open("", "w") do |output|
         # Match encoding of output string to that of input string
@@ -444,29 +460,29 @@ module GitCommitNotifier
         output.string
       end
     end
-    
+
+    # Commit to link mapping.
+    COMMIT_LINK_MAP = {
+      :gitweb    => lambda { |config, commit| "<a href='#{config['gitweb']['path']}?p=#{config['gitweb']['project'] || "#{Git.repo_name}.git"};a=commitdiff;h=#{commit}'>#{commit}</a>" },
+      :gitorious => lambda { |config, commit| "<a href='#{config['gitorious']['path']}/#{config['gitorious']['project']}/#{config['gitorious']['repository']}/commit/#{commit}'>#{commit}</a>" },
+      :trac      => lambda { |config, commit| "<a href='#{config['trac']['path']}/#{commit}'>#{commit}</a>" },
+      :cgit      => lambda { |config, commit| "<a href='#{config['cgit']['path']}/#{config['cgit']['project']}/commit/?id=#{commit}'>#{commit}</a>" },
+      :gitlabhq  => lambda { |config, commit| "<a href='#{config['gitlabhq']['path']}/#{Git.repo_name.gsub(".", "_")}/commits/#{commit}'>#{commit}</a>" },
+      :redmine   => lambda { |config, commit| "<a href='#{config['redmine']['path']}/projects/#{config['redmine']['project'] || Git.repo_name}/repository/revisions/#{commit}'>#{commit}</a>" },
+      :default   => lambda { |config, commit| commit.to_s }
+    }.freeze
+
+    # Gets HTML markup for specified commit.
+    # @param [String] commit Unique identifier of commit.
+    # @return [String] HTML markup for specified commit.
+    # @see COMMIT_LINK_MAP
     def markup_commit_for_html(commit)
-      commit = if config["link_files"]
-        if config["link_files"] == "gitweb" && config["gitweb"]
-          "<a href='#{config['gitweb']['path']}?p=#{Git.repo_name}.git;a=commitdiff;h=#{commit}'>#{commit}</a>"
-        elsif config["link_files"] == "gitorious" && config["gitorious"]
-          "<a href='#{config['gitorious']['path']}/#{config['gitorious']['project']}/#{config['gitorious']['repository']}/commit/#{commit}'>#{commit}</a>"
-        elsif config["link_files"] == "trac" && config["trac"]
-          "<a href='#{config['trac']['path']}/#{commit}'>#{commit}</a>"
-        elsif config["link_files"] == "cgit" && config["cgit"]
-          "<a href='#{config['cgit']['path']}/#{config['cgit']['project']}/commit/?id=#{commit}'>#{commit}</a>"
-        elsif config["link_files"] == "gitlabhq" && config["gitlabhq"]
-          "<a href='#{config['gitlabhq']['path']}/#{Git.repo_name.gsub(".", "_")}/commits/#{commit}'>#{commit}</a>"
-        elsif config["link_files"] == "redmine" && config["redmine"]
-          "<a href='#{config['redmine']['path']}/projects/#{config['redmine']['project'] || Git.repo_name}/repository/revisions/#{commit}'>#{commit}</a>"
-        else
-          "#{commit}"
-        end
-      else
-        "#{commit}"
-      end
+      mode = (config["link_files"] || "default").to_sym
+      mode = :default  unless config.has_key?(mode)
+      mode = :default  unless COMMIT_LINK_MAP.has_key?(mode)
+      COMMIT_LINK_MAP[mode].call(config, commit)
     end
-    
+
     def diff_for_commit(commit)
       @current_commit = commit
       raw_diff = truncate_long_lines(Git.show(commit, :ignore_whitespaces => ignore_whitespaces?))
@@ -489,19 +505,19 @@ module GitCommitNotifier
       title = "<dl class=\"title\">"
       title += "<dt>Commit</dt><dd>#{markup_commit_for_html(commit_info[:commit])}</dd>\n"
       title += "<dt>Branch</dt><dd>#{CGI.escapeHTML(branch_name)}</dd>\n" if branch_name
-      
+
       title += "<dt>Author</dt><dd>#{CGI.escapeHTML(commit_info[:author])} &lt;#{commit_info[:email]}&gt;</dd>\n"
-      
+
       # Show separate committer name/email only if it differs from author
       if commit_info[:author] != commit_info[:committer] || commit_info[:email] != commit_info[:commit_email]
         title += "<dt>Committer</dt><dd>#{CGI.escapeHTML(commit_info[:committer])} &lt;#{commit_info[:commit_email]}&gt;</dd>\n"
       end
 
       title += "<dt>Date</dt><dd>#{CGI.escapeHTML commit_info[:date]}</dd>\n"
-      
+
       multi_line_message = commit_info[:message].count > 1
       title += "<dt>Message</dt><dd class='#{multi_line_message ? "multi-line" : ""}'>#{message_array_as_html(commit_info[:message])}</dd>\n"
-      
+
       title += "</dl>"
 
       text = "#{raw_diff}"
@@ -521,38 +537,38 @@ module GitCommitNotifier
     end
 
     def diff_for_lightweight_tag(tag, rev, change_type)
-    
+
       if change_type == :delete
         message = "Remove Lightweight Tag #{tag}"
-        
+
         html = "<dl class='title'>"
         html += "<dt>Tag</dt><dd>#{CGI.escapeHTML(tag)} (removed)</dd>\n"
         html += "<dt>Type</dt><dd>lightweight</dd>\n"
         html += "<dt>Commit</dt><dd>#{markup_commit_for_html(rev)}</dd>\n"
         html += "</dl>"
-        
+
         text = "Remove Tag: #{tag}\n"
         text += "Type: lightweight\n"
         text += "Commit: #{rev}\n"
       else
         message = "#{change_type == :create ? "Add" : "Update"} Lightweight Tag #{tag}"
-        
+
         html = "<dl class='title'>"
         html += "<dt>Tag</dt><dd>#{CGI.escapeHTML(tag)} (#{change_type == :create ? "added" : "updated"})</dd>\n"
         html += "<dt>Type</dt><dd>lightweight</dd>\n"
         html += "<dt>Commit</dt><dd>#{markup_commit_for_html(rev)}</dd>\n"
         html += "</dl>"
-        
+
         text = "Tag: #{tag} (#{change_type == :create ? "added" : "updated"})\n"
         text += "Type: lightweight\n"
         text += "Commit: #{rev}\n"
       end
-      
+
       commit_info = {
         :commit => rev,
         :message => message
       }
-      
+
       @result << {
         :commit_info => commit_info,
         :html_content => html,
@@ -561,26 +577,26 @@ module GitCommitNotifier
     end
 
     def diff_for_annotated_tag(tag, rev, change_type)
-    
+
       commit_info = {
         :commit => rev
       }
-      
+
       if change_type == :delete
         message = "Remove Annotated Tag #{tag}"
-        
+
         html = "<dl class='title'>"
         html += "<dt>Tag</dt><dd>#{CGI.escapeHTML(tag)} (removed)</dd>\n"
         html += "<dt>Type</dt><dd>annotated</dd>\n"
         html += "</dl>"
-        
+
         text = message
         commit_info[:message] = message
       else
         tag_info = Git.tag_info(ref_name)
 
         message = tag_info[:subject] || "#{change_type == :create ? "Add" : "Update"} Annotated Tag #{tag}"
-        
+
         html = "<dl class='title'>"
         html += "<dt>Tag</dt><dd>#{CGI.escapeHTML(tag)} (#{change_type == :create ? "added" : "updated"})</dd>\n"
         html += "<dt>Type</dt><dd>annotated</dd>\n"
@@ -591,17 +607,17 @@ module GitCommitNotifier
         multi_line_message = message_array.count > 1
         html += "<dt>Message</dt><dd class='#{multi_line_message ? "multi-line" : ""}'>#{message_array_as_html(message_array)}</dd>\n"
         html += "</dl>"
-        
+
         text = "Tag:</strong> #{tag} (#{change_type == :create ? "added" : "updated"})\n"
         text += "Type: annotated\n"
         text += "Commit: #{tag_info[:tagobject]}\n"
         text += "Tagger: tag_info[:taggername] tag_info[:taggeremail]\n"
         text += "Message: #{tag_info[:contents]}\n"
-        
+
         commit_info[:message] = message
         commit_info[:author], commit_info[:email] = author_name_and_email("#{tag_info[:taggername]} #{tag_info[:taggeremail]}")
       end
-      
+
       @result << {
         :commit_info => commit_info,
         :html_content => html,
@@ -609,7 +625,7 @@ module GitCommitNotifier
       }
     end
 
-    def diff_for_branch(branch, rev, change_type)      
+    def diff_for_branch(branch, rev, change_type)
       commits = case change_type
       when :delete
         puts "ignoring branch delete"
@@ -617,11 +633,11 @@ module GitCommitNotifier
       when :create, :update
         # Note that "unique_commits_per_branch" really means "consider commits
         # on this branch without regard to whether they occur on other branches"
-        # The flag unique_to_current_branch passed to new_commits means the 
+        # The flag unique_to_current_branch passed to new_commits means the
         # opposite: "consider only commits that are unique to this branch"
         Git.new_commits(oldrev, newrev, ref_name, !unique_commits_per_branch?)
       end
-      
+
       # Add each diff to @result
       commits.each do |commit|
           commit_result = diff_for_commit(commit)
@@ -636,7 +652,7 @@ module GitCommitNotifier
 
     def diff_between_revisions(rev1, rev2, repo, ref_name)
       clear_result
-      
+
       # Cleanup revs
       @oldrev = Git.rev_parse(rev1)
       @newrev = Git.rev_parse(rev2)
@@ -650,7 +666,7 @@ module GitCommitNotifier
       else
         :update
       end
-      
+
       # Establish type of the revs
       @oldrev_type = Git.rev_type(@oldrev)
       @newrev_type = Git.rev_type(@newrev)
@@ -661,7 +677,7 @@ module GitCommitNotifier
         @rev_type = @newrev_type
         @rev = @newrev
       end
-      
+
       # Determine what to do based on the ref_name and the rev_type
       case "#{@ref_name},#{@rev_type}"
       when %r!^refs/tags/(.+),commit$!
@@ -680,7 +696,7 @@ module GitCommitNotifier
         # Something we don't understand
         puts "Unknown change type #{ref_name},#{@rev_type}"
       end
-      
+
       # If a block was given, pass it the results, in turn
       @result.each { |result| yield result }  if block_given?
     end
@@ -698,7 +714,7 @@ module GitCommitNotifier
     end
 
     def do_message_integration(message)
-      return message unless config['message_integration'].respond_to?(:each_pair)
+      return message  unless config['message_integration'].respond_to?(:each_pair)
       config['message_integration'].each_pair do |pm, url|
         pm_def = DiffToHtml::INTEGRATION_MAP[pm.to_sym] or next
         search_for = pm_def[:search_for]
@@ -711,7 +727,7 @@ module GitCommitNotifier
     end
 
     def do_message_map(message)
-      return message unless config['message_map'].respond_to?(:each_pair)
+      return message  unless config['message_map'].respond_to?(:each_pair)
       config['message_map'].each_pair do |search_for, replace_with|
         message_replace!(message, Regexp.new(search_for), replace_with)
       end
@@ -723,3 +739,4 @@ module GitCommitNotifier
     end
   end
 end
+