Skip to content

Commit

Permalink
Merge pull request #80 from sopel-irc/fix-self-assign-extra-space
Browse files Browse the repository at this point in the history
formatting: fix double space in webhook output for self-assignments
  • Loading branch information
dgw authored Feb 3, 2021
2 parents 2c921ef + d33f6ae commit 273bac6
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions sopel_modules/github/formatting.py
Original file line number Diff line number Diff line change
Expand Up @@ -325,15 +325,18 @@ def fmt_issue_title_edit(payload=None):
def fmt_issue_assignee_message(payload=None):
if not payload:
payload = current_payload

target = ''
assignee = payload['assignee']['login']
self_assign = False
if (payload['assignee']['login'] == payload['sender']['login']):

if assignee == payload['sender']['login']:
self_assign = True
else:
target = 'to ' if payload['action'] == 'assigned' else 'from '
target = target + fmt_name(payload['assignee']['login'])
return '[{}] {} {}{} {} #{} {} ({})'.format(
prep = 'to' if payload['action'] == 'assigned' else 'from'
target = ' {} {}'.format(prep, fmt_name(assignee))

return '[{}] {} {}{} {} #{}{} ({})'.format(
fmt_repo(payload['repository']['name']),
fmt_name(payload['sender']['login']),
'self-' if self_assign else '',
Expand Down

0 comments on commit 273bac6

Please sign in to comment.