From b248a47658a693757a8355cd8c2c93dd51f94d06 Mon Sep 17 00:00:00 2001 From: "Brian S. Stephan" Date: Thu, 8 Sep 2016 17:36:11 -0500 Subject: [PATCH] gitlab: evenly distribute MRs among reviewers bss/dr.botzo#8 --- 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 edecdea..63209b9 100644 --- a/dr_botzo/gitlab_bot/lib.py +++ b/dr_botzo/gitlab_bot/lib.py @@ -193,7 +193,7 @@ class GitlabBot(object): if merge_request.assignee is None or merge_request.assignee.username not in review_candidates: if len(review_candidates) > 0: - new_reviewer = random.choice(review_candidates) + new_reviewer = review_candidates[merge_request.iid % len(review_candidates)] log.debug("%s is the new reviewer", new_reviewer) # get the user object for the new reviewer @@ -219,7 +219,7 @@ class GitlabBot(object): if merge_request.assignee is None or merge_request.assignee.username not in accept_candidates: if len(accept_candidates) > 0: - new_accepter = random.choice(accept_candidates) + new_accepter = accept_candidates[merge_request.iid % len(accept_candidates)] log.debug("%s is the new accepter", new_accepter) # get the user object for the new accepter