Create MR with reviewers #1267

Activity

1. JoelFeiner commented on Jan 26, 2021

   

   JoelFeiner

   on Jan 26, 2021

   More actions

   @mary3000 I see reviewer_ids in the API. Perhaps the documentation just hadn't been updated yet?
2. m00lecule commented on Feb 9, 2021

   

   m00lecule

   on Feb 9, 2021

   Last edited by m00lecule

   More actions

   Hello! I am also struggling to apply following in python-gitlab 2.6.0

       mr = project.mergerequests.list(state="opened", wip="no", ordered_by="updated_at", sort="asc", per_page=999)[0]
       review_group = gl.groups.get(gitlab_review_group_name)
       members = review_group.members.list()
       mr.reviewer_ids = [members[0].get_id()]
       mr.save()

   Result:

   Traceback (most recent call last):
       mr.save()
     File "/XXXXXXXXXX/.venv/lib/python3.6/site-packages/gitlab/mixins.py", line 385, in save
       server_data = self.manager.update(obj_id, updated_data, **kwargs)
     File "XXXXXXXXXXXXX/.venv/lib/python3.6/site-packages/gitlab/exceptions.py", line 281, in wrapped_f
       raise error(e.error_message, e.response_code, e.response_body) from e
   gitlab.exceptions.GitlabUpdateError: 400: assignee_id, assignee_ids, description, labels, add_labels, remove_labels, milestone_id, remove_source_branch, allow_collaboration, allow_maintainer_to_push, squash, target_branch, title, state_event, discussion_locked are missing, at least one parameter must be provided

   Edit:

   This feature has not been added yet - link
3. mattiasb commented on Feb 27, 2021

   

   mattiasb

   on Feb 27, 2021

   More actions

   This should be in now since a couple of days. See here
4. spyoungtech commented on Apr 13, 2021

   

   spyoungtech

   on Apr 13, 2021

   Contributor

   More actions

   It would also be nice to be able to change the reviewers on an existing merge request.
5. spyoungtech commented on Apr 13, 2021

   

   spyoungtech

   on Apr 13, 2021

   Last edited by spyoungtech

   Contributor

   More actions

   As a workaround for this, you can do the following:

   # update the reviewers on the MR
   updated_data = mr.manager.update(mr.get_id(), {'reviewer_ids': [1,2,3]})
   
   #  for completeness, update the object from the server response
   if updated_data:
       mr._update_attrs(updated_data)
6. mykolasolodukha commented on Aug 8, 2022

   

   mykolasolodukha

   on Aug 8, 2022

   More actions

   This might've been closed now since the #1396 is merged.
7. nejch commented on Aug 9, 2022

   

   nejch

   on Aug 9, 2022

   Member

   More actions

   You're right @mykolasolodukha, this does already work. Thanks for the catch!

   mr = project.mergerequests.create(
       {
           "source_branch": "your-branch",
           "target_branch": "main",
           "title": "your MR",
           "reviewer_ids": [reviewer_id]
       }
   )
   
   mr.reviewer_ids = [reviewer_id, another_reviewer_id]
   mr.save()
8. nejch

   closed this as completedon Aug 9, 2022
9. github-actions

   locked as resolved and limited conversation to collaborators on Aug 14, 2023