Fix formatting

Author: Adrian Huber

hcloud/placement_groups/client.py

@@ -39,17 +39,21 @@ def get_by_id(self, id):
         :param id: int
         :return: :class:`BoundPlacementGroup <hcloud.placement_groups.client.BoundPlacementGroup>`
         """
-        response = self._client.request(url="/placement_groups/{placement_group_id}".format(placement_group_id=id), method="GET")
-        return BoundPlacementGroup(self, response['placement_group'])
-
-    def get_list(self,
-                 label_selector=None,  # type: Optional[str]
-                 page=None,  # type: Optional[int]
-                 per_page=None,  # type: Optional[int]
-                 name=None,  # type: Optional[str]
-                 sort=None,  # type: Optional[List[str]]
-                 type=None,  # type: Optional[str]
-                 ):
+        response = self._client.request(
+            url="/placement_groups/{placement_group_id}".format(placement_group_id=id),
+            method="GET",
+        )
+        return BoundPlacementGroup(self, response["placement_group"])
+
+    def get_list(
+        self,
+        label_selector=None,  # type: Optional[str]
+        page=None,  # type: Optional[int]
+        per_page=None,  # type: Optional[int]
+        name=None,  # type: Optional[str]
+        sort=None,  # type: Optional[List[str]]
+        type=None,  # type: Optional[str]
+    ):
         # type: (...) -> PageResults[List[BoundPlacementGroup]]
         """Get a list of Placement Groups
 
@@ -69,19 +73,24 @@ def get_list(self,
         params = {}
 
         if label_selector is not None:
-            params['label_selector'] = label_selector
+            params["label_selector"] = label_selector
         if page is not None:
-            params['page'] = page
+            params["page"] = page
         if per_page is not None:
-            params['per_page'] = per_page
+            params["per_page"] = per_page
         if name is not None:
-            params['name'] = name
+            params["name"] = name
         if sort is not None:
-            params['sort'] = sort
+            params["sort"] = sort
         if type is not None:
-            params['type'] = type
-        response = self._client.request(url="/placement_groups", method="GET", params=params)
-        placement_groups = [BoundPlacementGroup(self, placement_group_data) for placement_group_data in response['placement_groups']]
+            params["type"] = type
+        response = self._client.request(
+            url="/placement_groups", method="GET", params=params
+        )
+        placement_groups = [
+            BoundPlacementGroup(self, placement_group_data)
+            for placement_group_data in response["placement_groups"]
+        ]
 
         return self._add_meta_to_result(placement_groups, response)
 
@@ -97,7 +106,9 @@ def get_all(self, label_selector=None, name=None, sort=None):
                Choices: id name created (You can add one of ":asc", ":desc" to modify sort order. ( ":asc" is default))
         :return: List[:class:`BoundPlacementGroup <hcloud.placement_groups.client.BoundPlacementGroup>`]
         """
-        return super(PlacementGroupsClient, self).get_all(label_selector=label_selector, name=name, sort=sort)
+        return super(PlacementGroupsClient, self).get_all(
+            label_selector=label_selector, name=name, sort=sort
+        )
 
     def get_by_name(self, name):
         # type: (str) -> BoundPlacementGroup
@@ -109,11 +120,12 @@ def get_by_name(self, name):
         """
         return super(PlacementGroupsClient, self).get_by_name(name)
 
-    def create(self,
-               name,  # type: str
-               type,  # type: str
-               labels=None,  # type: Optional[Dict[str, str]]
-               ):
+    def create(
+        self,
+        name,  # type: str
+        type,  # type: str
+        labels=None,  # type: Optional[Dict[str, str]]
+    ):
         # type: (...) -> CreatePlacementGroupResponse
         """Creates a new Placement Group.
 
@@ -126,21 +138,20 @@ def create(self,
 
         :return: :class:`CreatePlacementGroupResponse <hcloud.placement_groups.domain.CreatePlacementGroupResponse>`
         """
-        data = {
-            'name': name,
-            'type': type
-        }
+        data = {"name": name, "type": type}
         if labels is not None:
-            data['labels'] = labels
-        response = self._client.request(url="/placement_groups", json=data, method="POST")
+            data["labels"] = labels
+        response = self._client.request(
+            url="/placement_groups", json=data, method="POST"
+        )
 
         action = None
-        if response.get('action') is not None:
-            action = BoundAction(self._client.action, response['action'])
+        if response.get("action") is not None:
+            action = BoundAction(self._client.action, response["action"])
 
         result = CreatePlacementGroupResponse(
-            placement_group=BoundPlacementGroup(self, response['placement_group']),
-            action=action
+            placement_group=BoundPlacementGroup(self, response["placement_group"]),
+            action=action,
         )
         return result
 
@@ -158,13 +169,18 @@ def update(self, placement_group, labels=None, name=None):
 
         data = {}
         if labels is not None:
-            data['labels'] = labels
+            data["labels"] = labels
         if name is not None:
-            data['name'] = name
-
-        response = self._client.request(url="/placement_groups/{placement_group_id}".format(placement_group_id=placement_group.id),
-                                        method="PUT", json=data)
-        return BoundPlacementGroup(self, response['placement_group'])
+            data["name"] = name
+
+        response = self._client.request(
+            url="/placement_groups/{placement_group_id}".format(
+                placement_group_id=placement_group.id
+            ),
+            method="PUT",
+            json=data,
+        )
+        return BoundPlacementGroup(self, response["placement_group"])
 
     def delete(self, placement_group):
         # type: (PlacementGroup) -> bool
@@ -173,6 +189,10 @@ def delete(self, placement_group):
         :param placement_group: :class:`BoundPlacementGroup <hcloud.placement_groups.client.BoundPlacementGroup>` or :class:`PlacementGroup <hcloud.placement_groups.domain.PlacementGroup>`
         :return: boolean
         """
-        self._client.request(url="/placement_groups/{placement_group_id}".format(placement_group_id=placement_group.id),
-                             method="DELETE")
+        self._client.request(
+            url="/placement_groups/{placement_group_id}".format(
+                placement_group_id=placement_group.id
+            ),
+            method="DELETE",
+        )
         return True

hcloud/placement_groups/domain.py

@@ -21,28 +21,15 @@ class PlacementGroup(BaseDomain):
            Point in time when the image was created
     """
 
-    __slots__ = (
-        "id",
-        "name",
-        "labels",
-        "servers",
-        "type",
-        "created"
-    )
+    __slots__ = ("id", "name", "labels", "servers", "type", "created")
 
     """Placement Group type spread
        spreads all servers in the group on different vhosts
     """
     TYPE_SPREAD = "spread"
 
     def __init__(
-        self,
-        id=None,
-        name=None,
-        labels=None,
-        servers=None,
-        type=None,
-        created=None
+        self, id=None, name=None, labels=None, servers=None, type=None, created=None
     ):
         self.id = id
         self.name = name
@@ -60,15 +47,13 @@ class CreatePlacementGroupResponse(BaseDomain):
     :param action: :class:`BoundAction <hcloud.actions.client.BoundAction>`
            The Action which shows the progress of the Placement Group Creation
     """
-    __slots__ = (
-        "placement_group",
-        "action"
-    )
+
+    __slots__ = ("placement_group", "action")
 
     def __init__(
-            self,
-            placement_group,  # type: BoundPlacementGroup
-            action,  # type: BoundAction
+        self,
+        placement_group,  # type: BoundPlacementGroup
+        action,  # type: BoundAction
     ):
         self.placement_group = placement_group
         self.action = action

hcloud/servers/client.py

@@ -104,8 +104,10 @@ def __init__(self, client, data, complete=True):
 
         placement_group = data.get("placement_group")
         if placement_group:
-            placement_group = BoundPlacementGroup(client._client.placement_groups, placement_group)
-            data['placement_group'] = placement_group
+            placement_group = BoundPlacementGroup(
+                client._client.placement_groups, placement_group
+            )
+            data["placement_group"] = placement_group
 
         super(BoundServer, self).__init__(client, data, complete)
 
@@ -462,22 +464,23 @@ def get_by_name(self, name):
         """
         return super(ServersClient, self).get_by_name(name)
 
-    def create(self,
-               name,  # type: str
-               server_type,  # type: ServerType
-               image,  # type: Image
-               ssh_keys=None,  # type: Optional[List[SSHKey]]
-               volumes=None,  # type: Optional[List[Volume]]
-               firewalls=None,  # type: Optional[List[Firewall]]
-               networks=None,  # type: Optional[List[Network]]
-               user_data=None,  # type: Optional[str]
-               labels=None,  # type: Optional[Dict[str, str]]
-               location=None,  # type: Optional[Location]
-               datacenter=None,  # type: Optional[Datacenter]
-               start_after_create=True,  # type: Optional[bool]
-               automount=None,  # type: Optional[bool]
-               placement_group=None  # type: Optional[PlacementGroup]
-               ):
+    def create(
+        self,
+        name,  # type: str
+        server_type,  # type: ServerType
+        image,  # type: Image
+        ssh_keys=None,  # type: Optional[List[SSHKey]]
+        volumes=None,  # type: Optional[List[Volume]]
+        firewalls=None,  # type: Optional[List[Firewall]]
+        networks=None,  # type: Optional[List[Network]]
+        user_data=None,  # type: Optional[str]
+        labels=None,  # type: Optional[Dict[str, str]]
+        location=None,  # type: Optional[Location]
+        datacenter=None,  # type: Optional[Datacenter]
+        start_after_create=True,  # type: Optional[bool]
+        automount=None,  # type: Optional[bool]
+        placement_group=None,  # type: Optional[PlacementGroup]
+    ):
         # type: (...) -> CreateServerResponse
         """Creates a new server. Returns preliminary information about the server as well as an action that covers progress of creation.
 
@@ -533,7 +536,7 @@ def create(self,
         if automount is not None:
             data["automount"] = automount
         if placement_group is not None:
-            data['placement_group'] = placement_group.id
+            data["placement_group"] = placement_group.id
 
         response = self._client.request(url="/servers", method="POST", json=data)
 
@@ -1014,9 +1017,13 @@ def change_alias_ips(self, server, network, alias_ips):
         """
         data = {"network": network.id, "alias_ips": alias_ips}
         response = self._client.request(
-            url="/servers/{server_id}/actions/change_alias_ips".format(server_id=server.id), method="POST",
-            json=data)
-        return BoundAction(self._client.actions, response['action'])
+            url="/servers/{server_id}/actions/change_alias_ips".format(
+                server_id=server.id
+            ),
+            method="POST",
+            json=data,
+        )
+        return BoundAction(self._client.actions, response["action"])
 
     def add_to_placement_group(self, server, placement_group):
         # type: (Union[Server,BoundServer], Union[PlacementGroup,BoundPlacementGroup]) -> BoundAction
@@ -1030,9 +1037,13 @@ def add_to_placement_group(self, server, placement_group):
             "placement_group": str(placement_group.id),
         }
         response = self._client.request(
-            url="/servers/{server_id}/actions/add_to_placement_group".format(server_id=server.id), method="POST",
-            json=data)
-        return BoundAction(self._client.actions, response['action'])
+            url="/servers/{server_id}/actions/add_to_placement_group".format(
+                server_id=server.id
+            ),
+            method="POST",
+            json=data,
+        )
+        return BoundAction(self._client.actions, response["action"])
 
     def remove_from_placement_group(self, server):
         # type: (Union[Server,BoundServer]) -> BoundAction
@@ -1042,5 +1053,9 @@ def remove_from_placement_group(self, server):
         :return: :class:`BoundAction <hcloud.actions.client.BoundAction>`
         """
         response = self._client.request(
-            url="/servers/{server_id}/actions/remove_from_placement_group".format(server_id=server.id), method="POST")
-        return BoundAction(self._client.actions, response['action'])
+            url="/servers/{server_id}/actions/remove_from_placement_group".format(
+                server_id=server.id
+            ),
+            method="POST",
+        )
+        return BoundAction(self._client.actions, response["action"])

hcloud/servers/domain.py

@@ -84,32 +84,32 @@ class Server(BaseDomain):
         "private_net",
         "created",
         "primary_disk_size",
-        "placement_group"
+        "placement_group",
     )
 
     def __init__(
-            self,
-            id,
-            name=None,
-            status=None,
-            created=None,
-            public_net=None,
-            server_type=None,
-            datacenter=None,
-            image=None,
-            iso=None,
-            rescue_enabled=None,
-            locked=None,
-            backup_window=None,
-            outgoing_traffic=None,
-            ingoing_traffic=None,
-            included_traffic=None,
-            protection=None,
-            labels=None,
-            volumes=None,
-            private_net=None,
-            primary_disk_size=None,
-            placement_group=None,
+        self,
+        id,
+        name=None,
+        status=None,
+        created=None,
+        public_net=None,
+        server_type=None,
+        datacenter=None,
+        image=None,
+        iso=None,
+        rescue_enabled=None,
+        locked=None,
+        backup_window=None,
+        outgoing_traffic=None,
+        ingoing_traffic=None,
+        included_traffic=None,
+        protection=None,
+        labels=None,
+        volumes=None,
+        private_net=None,
+        primary_disk_size=None,
+        placement_group=None,
     ):
         self.id = id
         self.name = name