style: fix style

robinson96 · robinson96 · commit 3586b693f62d · 2020-09-30T08:12:47.000-07:00
diff --git a/gitlab/v4/objects/__init__.py b/gitlab/v4/objects/__init__.py
@@ -2996,7 +2996,12 @@ class ProjectMergeRequestApprovalManager(GetWithoutIdMixin, UpdateMixin, RESTMan
 
     @exc.on_http_error(exc.GitlabUpdateError)
     def set_approvers(
-        self, approvals_required, approver_ids=None, approver_group_ids=None, approval_rule_name="name", **kwargs
+        self,
+        approvals_required,
+        approver_ids=None,
+        approver_group_ids=None,
+        approval_rule_name="name",
+        **kwargs
     ):
         """Change MR-level allowed approvers and approver groups.
 
@@ -3050,27 +3055,33 @@ def save(self, **kwargs):
             GitlabAuthenticationError: If authentication is not correct
             GitlabUpdateError: If the server cannot perform the request
         """
-        #There is a mismatch between the name of our id attribute and the put REST API name for the
-        #project_id, so we override it here.
+        # There is a mismatch between the name of our id attribute and the put REST API name for the
+        # project_id, so we override it here.
         self.approval_rule_id = self.id
         self.merge_request_iid = self._parent_attrs["mr_iid"]
         self.id = self._parent_attrs["project_id"]
-        #save will update self.id with the result from the server, so no need to overwrite with 
-        #what it was before we overwrote it."""
+        # save will update self.id with the result from the server, so no need to overwrite with
+        # what it was before we overwrote it."""
         SaveMixin.save(self, **kwargs)
 
 
-class ProjectMergeRequestApprovalRuleManger(ListMixin, UpdateMixin, CreateMixin, RESTManager):
+class ProjectMergeRequestApprovalRuleManger(
+    ListMixin, UpdateMixin, CreateMixin, RESTManager
+):
     _path = "/projects/%(project_id)s/merge_requests/%(mr_iid)s/approval_rules"
     _obj_cls = ProjectMergeRequestApprovalRule
     _from_parent_attrs = {"project_id": "project_id", "mr_iid": "iid"}
-    _list_filters = ("name","rule_type")
-    _update_attrs = (("id","merge_request_iid","approval_rule_id","name","approvals_required"),
-                    ("user_ids","group_ids")) 
-    #Important: When approval_project_rule_id is set, the name, users and groups of
-    #project-level rule will be copied. The approvals_required specified will be used.  """
-    _create_attrs = (("id","merge_request_iid","name","approvals_required"),
-                    ("approval_project_rule_id", "user_ids","group_ids"))
+    _list_filters = ("name", "rule_type")
+    _update_attrs = (
+        ("id", "merge_request_iid", "approval_rule_id", "name", "approvals_required"),
+        ("user_ids", "group_ids"),
+    )
+    # Important: When approval_project_rule_id is set, the name, users and groups of
+    # project-level rule will be copied. The approvals_required specified will be used.  """
+    _create_attrs = (
+        ("id", "merge_request_iid", "name", "approvals_required"),
+        ("approval_project_rule_id", "user_ids", "group_ids"),
+    )
 
     def create(self, data, **kwargs):
         """Create a new object.
