Firepup650/bitbot-3.11-fork
2
0
Fork 0
Code Issues Pull requests Activity Actions

Show commit message, not PR title, for PR pushes

Browse source
This commit is contained in:
jesopo 2019-06-26 15:13:01 +01:00
parent c5785a2d14
commit 24df8c996e
1 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
modules/git_webhooks/github.py
View file

@ -175,6 +175,9 @@ class GitHub(object):
def _flat_unique(self, commits, key): def _flat_unique(self, commits, key):
return set(itertools.chain(*(commit[key] for commit in commits))) 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): def push(self, full_name, data):
outputs = [] outputs = []
branch = data["ref"].split("/", 2)[2] branch = data["ref"].split("/", 2)[2]
@ -192,7 +195,7 @@ class GitHub(object):
for commit in data["commits"]: for commit in data["commits"]:
hash = commit["id"] hash = commit["id"]
hash_colored = utils.irc.color(self._short_hash(hash), colors.COLOR_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)) url = self._short_url(COMMIT_URL % (full_name, hash))
outputs.append( outputs.append(
@ -225,6 +228,8 @@ class GitHub(object):
branch = data["pull_request"]["base"]["ref"] branch = data["pull_request"]["base"]["ref"]
colored_branch = utils.irc.color(branch, colors.COLOR_BRANCH) colored_branch = utils.irc.color(branch, colors.COLOR_BRANCH)
title = data["pull_request"]["title"]
if action == "opened": if action == "opened":
action_desc = "requested %s merge into %s" % (number, action_desc = "requested %s merge into %s" % (number,
colored_branch) colored_branch)
@ -240,12 +245,12 @@ class GitHub(object):
action_desc = "marked %s ready for review" % number action_desc = "marked %s ready for review" % number
elif action == "synchronize": elif action == "synchronize":
action_desc = "committed to %s" % number 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"]) author = utils.irc.bold(data["sender"]["login"])
url = self._short_url(data["pull_request"]["html_url"]) url = self._short_url(data["pull_request"]["html_url"])
return ["[PR] %s %s: %s - %s" % ( 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): def pull_request_review(self, full_name, data):
if not data["action"] == "submitted": if not data["action"] == "submitted":