Add return types to all __str__ methods

docker/context/context.py

@@ -175,7 +175,7 @@ def remove(self):
     def __repr__(self):
         return f"<{self.__class__.__name__}: '{self.name}'>"
 
-    def __str__(self):
+    def __str__(self) -> str:
         return json.dumps(self.__call__(), indent=2)
 
     def __call__(self):

docker/errors.py

@@ -50,7 +50,7 @@ def __init__(self, message, response=None, explanation=None):
         self.response = response
         self.explanation = explanation
 
-    def __str__(self):
+    def __str__(self) -> str:
         message = super().__str__()
 
         if self.is_client_error():
@@ -121,7 +121,7 @@ class TLSParameterError(DockerException):
     def __init__(self, msg):
         self.msg = msg
 
-    def __str__(self):
+    def __str__(self) -> str:
         return self.msg + (". TLS configurations should map the Docker CLI "
                            "client configurations. See "
                            "https://docs.docker.com/engine/articles/https/ "
@@ -181,29 +181,29 @@ class MissingContextParameter(DockerException):
     def __init__(self, param):
         self.param = param
 
-    def __str__(self):
+    def __str__(self) -> str:
         return (f"missing parameter: {self.param}")
 
 
 class ContextAlreadyExists(DockerException):
     def __init__(self, name):
         self.name = name
 
-    def __str__(self):
+    def __str__(self) -> str:
         return (f"context {self.name} already exists")
 
 
 class ContextException(DockerException):
     def __init__(self, msg):
         self.msg = msg
 
-    def __str__(self):
+    def __str__(self) -> str:
         return (self.msg)
 
 
 class ContextNotFound(DockerException):
     def __init__(self, name):
         self.name = name
 
-    def __str__(self):
+    def __str__(self) -> str:
         return (f"context '{self.name}' not found")

docker/utils/proxy.py

@@ -68,7 +68,7 @@ def inject_proxy_environment(self, environment):
         # variables defined in "environment" to take precedence.
         return proxy_env + environment
 
-    def __str__(self):
+    def __str__(self) -> str:
         return (
             'ProxyConfig('
             f'http={self.http}, https={self.https}, '

scripts/versions.py

@@ -51,7 +51,7 @@ def order(self):
 
         return (int(self.major), int(self.minor), int(self.patch)) + stage
 
-    def __str__(self):
+    def __str__(self) -> str:
         stage = f'-{self.stage}' if self.stage else ''
         edition = f'-{self.edition}' if self.edition else ''
         return '.'.join(map(str, self[:3])) + edition + stage