Commit 46e656af authored by Simon Chabot's avatar Simon Chabot

fix: double check that an MR is not wip before assigning

it seems that filtering in the search is not enough, and some MR may be wip,
even when the 'wip="no"' parameter is given.

Let' skip WIP merge requests
parent 7da6382dfb12
Pipeline #19874 passed with stage
in 1 minute and 9 seconds
......@@ -162,6 +162,12 @@ class ReviewAssigner:
shuffle(merge_requests)
for merge_request in merge_requests:
if merge_request.work_in_progress:
# double check that the MR is not WIP.
# it seems that filtering in the search parameters is not enough
# XXX is it a Gitlab bug ?
continue
project = self.cnx.projects.get(merge_request.project_id)
if CLIENT_PROJECT_TAG in project.tag_list:
# it's a client project. Let's assume that someone will take
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment