feat(api): remove responsibility for API inconsistencies for MR reviewers

spyoungtech · web-flow · commit 3d985ee8cdd5 · 2021-06-16T12:43:14.000-07:00
diff --git a/gitlab/v4/objects/merge_requests.py b/gitlab/v4/objects/merge_requests.py
@@ -138,22 +138,6 @@ class ProjectMergeRequest(
 ):
     _id_attr = "iid"
 
-    @property
-    def reviewer_ids(self):
-        return [reviewer["id"] for reviewer in self.reviewers]
-
-    @reviewer_ids.setter
-    def reviewer_ids(self, new_reviewer_ids):
-        new_reviewers = [{"id": id} for id in set(new_reviewer_ids)]
-        new_reviewers.extend(
-            [
-                reviewer
-                for reviewer in self.reviewers
-                if reviewer["id"] in new_reviewer_ids
-            ]
-        )
-        self.reviewers = new_reviewers
-
     _managers = (
         ("approvals", "ProjectMergeRequestApprovalManager"),
         ("approval_rules", "ProjectMergeRequestApprovalRuleManager"),
