Added return value for 'ping', as I remember it's used in assertions …

…by users

pytransifex/api.py

@@ -249,12 +249,13 @@ def project_exists(self, project_slug: str) -> bool:
         )
 
     @ensure_login
-    def ping(self):
+    def ping(self) -> bool:
         """
         Exposing this just for the sake of satisfying qgis-plugin-cli's expectations
         There is no need to ping the server on the current implementation, as connection is handled by the SDK
         """
         logging.info("'ping' is deprecated!")
+        return True
 
     @ensure_login
     def get_project_stats(self, project_slug: str) -> dict[str, Any]:

pytransifex/interfaces.py

@@ -6,8 +6,7 @@ class Tx(ABC):
     # TODO
     # This interface modelled after api.py:Transifex satisfies the expectations of qgis-plugin-cli
     # but it falls short of requiring an implementation for methods that qgis-plugin-cli does not use:
-    #   coordinator, create_translation, delete_project, delete_resource, delete_team
-    # The question is whether I should implement them. Is there's a consumer downstream?
+    # { coordinator, create_translation, delete_project, delete_resource, delete_team }
 
     @abstractmethod
     def create_project(
@@ -77,5 +76,5 @@ def project_exists(self, project_slug: str) -> bool:
         raise NotImplementedError
 
     @abstractmethod
-    def ping(self):
+    def ping(self) -> bool:
         raise NotImplementedError