Issue 3681 - Add suport for "Fixes XXXX - ..." commit messages

sitescripts/hg/bin/update_issues.py

 # This file is part of Adblock Plus <https://adblockplus.org/>,
 # Copyright (C) 2006-2016 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 # GNU General Public License for more details.
 #
 # You should have received a copy of the GNU General Public License
 # along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
 
 """Hooks for integrating Mercurial with Trac.
 
 Update the issues that are referenced in commit messages when the commits
-are pushed and and "master" bookmark is moved. See README.md for more
+are pushed and `master` bookmark is moved. See README.md for more
 information on behavior and configuration.
 """
 
 import collections
 import contextlib
 import posixpath
 import re
 import xmlrpclib
 
 from sitescripts.utils import get_config, get_template
 
 
 _IssueRef = collections.namedtuple('IssueRef', 'id commits is_fixed')
 
 ISSUE_NUMBER_REGEX = re.compile(r'\b(issue|fixes)\s+(\d+)\b', re.I)
 NOISSUE_REGEX = re.compile(r'^noissue\b', re.I)
-COMMIT_MESSAGE_REGEX = re.compile('\[\S+\ ([^\]]+)\]')
+COMMIT_MESSAGE_REGEX = re.compile(r'\[\S+\ ([^\]]+)\]')
 
 
 @contextlib.contextmanager
 def _trac_proxy(ui, config, action_descr):
     trac_url = config.get('hg', 'trac_xmlrpc_url')
     try:
         yield xmlrpclib.ServerProxy(trac_url)
     except Exception as exc:
         if getattr(exc, 'faultCode', 0) == 404:
             ui.warn('warning: 404 (not found) while {}\n'.format(action_descr))
@@ skipping 30 matching lines @@
                                         'changes': ref.commits})
         with _trac_proxy(ui, config, 'getting issue {}'.format(ref.id)) as tp:
             attrs = tp.ticket.get(ref.id)[3]
             changes = {'_ts': attrs['_ts'], 'action': 'leave'}
             _update_issue(ui, config, ref.id, changes, comment_text)
 
 
 def _compile_module_regexps(ui, config, modules):
     for module, regexp in config.items('hg_module_milestones'):
         try:
-            yield module, re.compile('^' + regexp + '$', re.I)

[Sebastian Noack, 2016/05/17 09:10:57]

Don't use the plus operator when concatenating more than two strings, as per our
style guide. Use '^{}$'.format(regexp) instead.

[Vasily Kuznetsov, 2016/05/17 09:59:49]

Done.

+            yield module, re.compile('^{}$'.format(regexp), re.I)
         except Exception as exc:
             ui.warn('warning: skipped invalid regexp for module {} in '
                     "[hg_module_milestones] config: '{}' ({})\n"
                     .format(module, regexp, exc))
 
 
 def _get_module_milestones(ui, config, modules):
     module_regexps = dict(_compile_module_regexps(ui, config, modules))
     modules = module_regexps.keys()
     if not modules:
@@ skipping 51 matching lines @@
     commits_by_issue = collections.defaultdict(list)
     fixed_issues = set()
 
     for commit in commits:
         message = commit.description()
         if ' - ' not in message:
             ui.warn("warning: invalid commit message format: '{}'\n"
                     .format(message))
             continue
 
-        refs, rest = message.split(" - ", 1)
+        refs, rest = message.split(' - ', 1)
         issue_refs = ISSUE_NUMBER_REGEX.findall(refs)
         if issue_refs:
             for ref_type, issue_id in issue_refs:
                 issue_id = int(issue_id)
                 commits_by_issue[issue_id].append(commit)
                 if ref_type.lower() == 'fixes':
                     fixed_issues.add(issue_id)
         elif not NOISSUE_REGEX.search(refs):
             ui.warn("warning: no issue reference in commit message: '{}'\n"
                     .format(message))
 
     for issue_id, commits in sorted(commits_by_issue.items()):
         yield _IssueRef(issue_id, commits, is_fixed=issue_id in fixed_issues)
 
 
 def changegroup_hook(ui, repo, node, **kwargs):
     config = get_config()
     first_rev = repo[node].rev()
     pushed_commits = repo[first_rev:]
     refs = _collect_references(ui, pushed_commits)
     _post_comments(ui, repo, config, refs)
 
 
 def pushkey_hook(ui, repo, **kwargs):
     if (kwargs['namespace'] != 'bookmarks' or  # Not a bookmark move.
-            kwargs['key'] != 'master' or       # Not master bookmark.
+            kwargs['key'] != 'master' or       # Not `master` bookmark.
             not kwargs['old']):                # The bookmark is just created.
         return
 
     config = get_config()
     old_master_rev = repo[kwargs['old']].rev()
     new_master_rev = repo[kwargs['new']].rev()
     added_revs = repo.changelog.findmissingrevs([old_master_rev],
                                                 [new_master_rev])
     added_commits = [repo[rev] for rev in added_revs]
     refs = [ref for ref in _collect_references(ui, added_commits)
             if ref.is_fixed]
     _declare_fixed(ui, config, refs)
 
 
 # Alias for backward compatibility.
 hook = changegroup_hook