Skip to content

Commit 65ecadc

Browse files
author
Charles
committed
fix(objects): update set_approvers function call
Added a miss paramter update to the set_approvers function
1 parent 2cf12c7 commit 65ecadc

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

gitlab/v4/objects.py

+12-6
Original file line numberDiff line numberDiff line change
@@ -2685,7 +2685,9 @@ class ProjectMergeRequestApprovalManager(GetWithoutIdMixin, UpdateMixin, RESTMan
26852685
_update_uses_post = True
26862686

26872687
@exc.on_http_error(exc.GitlabUpdateError)
2688-
def set_approvers(self, approvals_required, approver_ids=None, approver_group_ids=None, **kwargs):
2688+
def set_approvers(
2689+
self, approvals_required, approver_ids=None, approver_group_ids=None, **kwargs
2690+
):
26892691
"""Change MR-level allowed approvers and approver groups.
26902692
26912693
Args:
@@ -2700,12 +2702,16 @@ def set_approvers(self, approvals_required, approver_ids=None, approver_group_id
27002702
approver_ids = approver_ids or []
27012703
approver_group_ids = approver_group_ids or []
27022704

2703-
path = "%s/%s/approval_rules" % (self._parent.manager.path, self._parent.get_id())
2705+
path = "%s/%s/approval_rules" % (
2706+
self._parent.manager.path,
2707+
self._parent.get_id(),
2708+
)
27042709
data = {
2705-
"approvals_required": approvals_required,
2706-
"rule_type": "regular",
2707-
"user_ids": approver_ids,
2708-
"group_ids": approver_group_ids}
2710+
"approvals_required": approvals_required,
2711+
"rule_type": "regular",
2712+
"user_ids": approver_ids,
2713+
"group_ids": approver_group_ids,
2714+
}
27092715
self.gitlab.http_put(path, post_data=data, **kwargs)
27102716

27112717

0 commit comments

Comments
 (0)