From 24df8c996e3718825fed5c5df77a5b21c7222fab Mon Sep 17 00:00:00 2001 From: jesopo Date: Wed, 26 Jun 2019 15:13:01 +0100 Subject: [PATCH] Show commit message, not PR title, for PR pushes --- modules/git_webhooks/github.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/modules/git_webhooks/github.py b/modules/git_webhooks/github.py index a5fd32ed..73a6b11a 100644 --- a/modules/git_webhooks/github.py +++ b/modules/git_webhooks/github.py @@ -175,6 +175,9 @@ class GitHub(object): def _flat_unique(self, commits, key): return set(itertools.chain(*(commit[key] for commit in commits))) + def _commit_message(self, message): + return message.split("\n")[0].strip() + def push(self, full_name, data): outputs = [] branch = data["ref"].split("/", 2)[2] @@ -192,7 +195,7 @@ class GitHub(object): for commit in data["commits"]: hash = commit["id"] hash_colored = utils.irc.color(self._short_hash(hash), colors.COLOR_ID) - message = commit["message"].split("\n")[0].strip() + message = self._commit_message(commit["message"]) url = self._short_url(COMMIT_URL % (full_name, hash)) outputs.append( @@ -225,6 +228,8 @@ class GitHub(object): branch = data["pull_request"]["base"]["ref"] colored_branch = utils.irc.color(branch, colors.COLOR_BRANCH) + title = data["pull_request"]["title"] + if action == "opened": action_desc = "requested %s merge into %s" % (number, colored_branch) @@ -240,12 +245,12 @@ class GitHub(object): action_desc = "marked %s ready for review" % number elif action == "synchronize": action_desc = "committed to %s" % number + title = self._commit_message(data["commit"]["message"]) - pr_title = data["pull_request"]["title"] author = utils.irc.bold(data["sender"]["login"]) url = self._short_url(data["pull_request"]["html_url"]) return ["[PR] %s %s: %s - %s" % ( - author, action_desc, pr_title, url)] + author, action_desc, title, url)] def pull_request_review(self, full_name, data): if not data["action"] == "submitted":