Merge lp:~dobey/tarmac/rm-commit-plug into lp:~ubuntuone-hackers/tarmac/trunk

[dobey]

Remove the broken commit plug-in.

This removes the plug-in which was added in https://code.launchpad.net/~james-w/tarmac/commit-message/+merge/140555 as it breaks the default commit message handling, isn't enabled via a config option, and duplicates functionality already in the commitmessage plug-in, but doesn't work with multiple reviewers. It also has no license header, or tests.

[Paul Hummer (rockstar)]

review: Approve

=== removed file 'tarmac/plugins/commit.py'
--- tarmac/plugins/commit.py	2012-12-18 20:45:26 +0000
+++ tarmac/plugins/commit.py	1970-01-01 00:00:00 +0000
@@ -1,24 +0,0 @@
-import re
-
-from tarmac.hooks import tarmac_hooks
-from tarmac.plugins import TarmacPlugin
-
-
-class CommitMessage(TarmacPlugin):
-
-    def run(self, command, target, source, proposal):
-        if not re.match(r'^\[.+?\].*$', proposal.commit_message):
-            params = ["[r=%s]" % proposal.reviewer.name]
-            for bug in proposal.source_branch.linked_bugs:
-                params.append("[bug=%s]" % bug.id)
-
-            proposal.commit_message = ("%s " % ",".join(params) +
-                                       proposal.commit_message)
-            self.logger.debug("Setting commit_message to %s",
-                              proposal.commit_message)
-        else:
-            self.logger.debug("%s has already right commit message",
-                              proposal)
-
-tarmac_hooks['tarmac_pre_commit'].hook(CommitMessage(),
-    'Commit messsage template editor.')