From 0b332f79c91cb863a88ee43f50aad4ce89a7d63e Mon Sep 17 00:00:00 2001 From: "Brian S. Stephan" Date: Fri, 1 Jul 2016 13:23:57 -0500 Subject: [PATCH] gitlab bot: use assignee name in comments when assigning a new user for review/acceptance, use their real name in the message, not their username (the reassign will include their username a split second later anyway) --- dr_botzo/gitlab_bot/lib.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dr_botzo/gitlab_bot/lib.py b/dr_botzo/gitlab_bot/lib.py index e7316a4..d4caf40 100644 --- a/dr_botzo/gitlab_bot/lib.py +++ b/dr_botzo/gitlab_bot/lib.py @@ -192,7 +192,7 @@ class GitlabBot(object): new_reviewer_obj = self.client.users.get_by_username(new_reviewer) # create note for the update - msg = {'body': self.random_new_reviewer_message().format(new_reviewer)} + msg = {'body': self.random_new_reviewer_message().format(new_reviewer_obj.name)} self.client.project_mergerequest_notes.create(msg, project_id=project_obj.id, merge_request_id=merge_request.id) @@ -218,7 +218,7 @@ class GitlabBot(object): new_accepter_obj = self.client.users.get_by_username(new_accepter) # create note for the update - msg = {'body': self.random_reviews_done_message().format(new_accepter)} + msg = {'body': self.random_reviews_done_message().format(new_accepter_obj.name)} self.client.project_mergerequest_notes.create(msg, project_id=project_obj.id, merge_request_id=merge_request.id)