Fixed missing annotations in src

Author: MrClock8163

pyproject.toml

@@ -68,3 +68,6 @@ version-file = "src/geocompy/_version.py"
 addopts = ["-ra", "-v", "--color=yes"]
 testpaths = ["tests"]
 required_plugins = "pytest-mock"
+
+[tool.mypy]
+strict = true

src/geocompy/communication.py

@@ -113,7 +113,7 @@ def is_open(self) -> bool:
         """
         raise NotImplementedError("interface does not implement 'is_open'")
 
-    def send(self, message: str):
+    def send(self, message: str) -> None:
         """
         Sends a single serialized message through the connection.
 
@@ -296,7 +296,7 @@ def __init__(
         if not self._port.is_open:
             self._port.open()
 
-    def __del__(self):
+    def __del__(self) -> None:
         self._port.close()
 
     def __enter__(self) -> SerialConnection:
@@ -307,10 +307,10 @@ def __exit__(
         exc_type: BaseException,
         exc_value: BaseException,
         exc_tb: TracebackType
-    ):
+    ) -> None:
         self._port.close()
 
-    def close(self):
+    def close(self) -> None:
         """
         Closes the serial port.
         """
@@ -328,7 +328,7 @@ def is_open(self) -> bool:
         """
         return self._port.is_open
 
-    def send(self, message: str):
+    def send(self, message: str) -> None:
         """
         Writes a single message to the serial line.
 

src/geocompy/data.py

@@ -34,8 +34,10 @@
     TypeAlias,
     Literal,
     Callable,
+    Iterator,
     TypeVar,
-    Self
+    Self,
+    Any
 )
 
 
@@ -758,7 +760,7 @@ def __str__(self) -> str:
     def __repr__(self) -> str:
         return str(self)
 
-    def __iter__(self):
+    def __iter__(self) -> Iterator[float]:
         return iter([self.x, self.y, self.z])
 
     def __getitem__(self, idx: int) -> float:
@@ -768,7 +770,7 @@ def __getitem__(self, idx: int) -> float:
         coords = (self.x, self.y, self.z)
         return coords[idx]
 
-    def __eq__(self, other) -> bool:
+    def __eq__(self, other: Any) -> bool:
         if type(other) is not type(self):
             return False
 

src/geocompy/protocols.py

@@ -179,7 +179,7 @@ class GeoComProtocol:
     Base class for GeoCom protocol versions.
 
     """
-    _R1P: re.Pattern = re.compile(
+    _R1P: re.Pattern[str] = re.compile(
         r"^%R1P,"
         r"(?P<comrc>\d+),"
         r"(?P<tr>\d+):"
@@ -241,7 +241,7 @@ def request(
         rpc: int,
         params: Iterable[int | float | bool | str | Angle | Byte] = (),
         parsers: Iterable[Callable[[str], Any]] | None = None
-    ) -> GeoComResponse[tuple]: ...
+    ) -> GeoComResponse[tuple[Any, ...]]: ...
 
     def request(
         self,
@@ -252,7 +252,7 @@ def request(
             | Callable[[str], Any]
             | None
         ) = None
-    ) -> GeoComResponse:
+    ) -> GeoComResponse[Any]:
         """
         Executes an RPC request and returns the parsed GeoCom response.
 
@@ -345,7 +345,7 @@ def parse_response(
         cmd: str,
         response: str,
         parsers: Iterable[Callable[[str], Any]] | None = None
-    ) -> GeoComResponse[tuple]: ...
+    ) -> GeoComResponse[tuple[Any, ...]]: ...
 
     def parse_response(
         self,
@@ -356,7 +356,7 @@ def parse_response(
             | Callable[[str], Any]
             | None
         ) = None
-    ) -> GeoComResponse:
+    ) -> GeoComResponse[Any]:
         """
         Parses RPC response and constructs :class:`GeoComResponse`
         instance.
@@ -402,7 +402,7 @@ def parse_response(
         elif not isinstance(parsers, Iterable):
             parsers = (parsers,)
 
-        params: list = []
+        params: list[Any] = []
         try:
             for func, value in zip(parsers, values.split(",")):
                 params.append(func(value))

src/geocompy/tps1100/csv.py

@@ -14,7 +14,7 @@
 """
 from __future__ import annotations
 
-from typing import Never
+from typing import Never, Any
 from typing_extensions import deprecated
 
 from ..data import enumparser
@@ -106,7 +106,8 @@ def get_user_instrument_name(self) -> Never:
     @deprecated("This command was removed for TPS1100 instruments")
     def set_user_instrument_name(
         self,
-        *args
+        *args: Any,
+        **kwargs: Any
     ) -> Never:
         """
         RPC 5005, ``CSV_SetUserInstrumentName``

src/geocompy/tps1100/edm.py

@@ -14,7 +14,7 @@
 """
 from __future__ import annotations
 
-from typing import Never
+from typing import Never, Any
 from typing_extensions import deprecated
 
 from ..data import (
@@ -88,7 +88,8 @@ def set_guidelight_intensity(
     @deprecated("This command was removed for TPS1100 instruments")
     def switch_edm(
         self,
-        *args
+        *args: Any,
+        **kwargs: Any
     ) -> Never:
         """
         RPC 1010, ``EDM_On``
@@ -117,7 +118,8 @@ def get_boomerang_filter_status(self) -> Never:
     @deprecated("This command was removed for TPS1100 instruments")
     def switch_boomerang_filter(
         self,
-        *args
+        *args: Any,
+        **kwargs: Any
     ) -> Never:
         """
         RPC 1007, ``EDM_SetBumerang``
@@ -147,7 +149,8 @@ def get_tracklight_brightness(self) -> Never:
     @deprecated("This command was removed for TPS1100 instruments")
     def set_tracklight_brightness(
         self,
-        *args
+        *args: Any,
+        **kwargs: Any
     ) -> Never:
         """
         RPC 1032, ``EDM_SetTrkLightBrightness``
@@ -178,7 +181,8 @@ def get_tracklight_status(self) -> Never:
     @deprecated("This command was removed for TPS1100 instruments")
     def switch_tracklight(
         self,
-        *args
+        *args: Any,
+        **kwargs: Any
     ) -> Never:
         """
         RPC 1031, ``EDM_SetTrkLightSwitch``

src/geocompy/tps1200p/aus.py

@@ -14,7 +14,7 @@
 """
 from __future__ import annotations
 
-from typing import Never
+from typing import Never, Any
 from typing_extensions import deprecated
 
 from ..tps1100.aus import TPS1100AUS
@@ -45,7 +45,8 @@ def get_rcs_search_status(self) -> Never:
     @deprecated("This command was removed for TPS1200 instruments")
     def switch_rcs_search(
         self,
-        *args
+        *args: Any,
+        **kwargs: Any
     ) -> Never:
         """
         RPC 18009, ``AUS_SwitchRcsSearch``

src/geocompy/tps1200p/aut.py

@@ -14,7 +14,7 @@
 """
 from __future__ import annotations
 
-from typing import Never
+from typing import Never, Any
 from typing_extensions import deprecated
 
 from ..data import toenum
@@ -49,7 +49,8 @@ def get_atr_status(self) -> Never:
     @deprecated("This command was removed for TPS1200 instruments")
     def switch_atr(
         self,
-        *args
+        *args: Any,
+        **kwargs: Any
     ) -> Never:
         """
         RPC 9018, ``AUT_SetATRStatus``
@@ -78,7 +79,8 @@ def get_lock_status(self) -> Never:
     @deprecated("This command was removed for TPS1200 instruments")
     def switch_lock(
         self,
-        *args
+        *args: Any,
+        **kwargs: Any
     ) -> Never:
         """
         RPC 9020, ``AUT_SetLockStatus``

src/geocompy/tps1200p/bap.py

@@ -14,7 +14,7 @@
 """
 from __future__ import annotations
 
-from typing import Never
+from typing import Never, Any
 from typing_extensions import deprecated
 
 from ..data import (
@@ -58,7 +58,8 @@ def get_last_displayed_error(self) -> Never:
     @deprecated("This command was removed for TPS1200 instruments")
     def set_prism_definition(
         self,
-        *args
+        *args: Any,
+        **kwargs: Any
     ) -> Never:
         """
         RPC 17024, ``BAP_SetPrismDef``

src/geocompy/tps1200p/com.py

@@ -14,7 +14,7 @@
 """
 from __future__ import annotations
 
-from typing import Never
+from typing import Never, Any
 from typing_extensions import deprecated
 
 from ..tps1100.com import TPS1100COM
@@ -32,7 +32,8 @@ class TPS1200PCOM(TPS1100COM):
     @deprecated("This command was removed for TPS1200 instruments")
     def set_send_delay(
         self,
-        *args
+        *args: Any,
+        **kwargs: Any
     ) -> Never:
         """
         RPC 109, ``COM_SetSendDelay``
@@ -48,7 +49,8 @@ def set_send_delay(
     @deprecated("This command was removed for TPS1200 instruments")
     def switch_signoff(
         self,
-        *args
+        *args: Any,
+        **kwargs: Any
     ) -> Never:
         """
         RPC 115, ``COM_EnableSignOff``