Merge pull request #9 from sabadam32/add_type_annotations_6

refactor: adds type annotations reported by mypy and updated pylint v…

Author: FoamyGuy

adafruit_ble_ibbq.py

@@ -20,6 +20,10 @@
 
 InkBird and EasyBBQ (from PyleUSA) are brands that use the iBBQ protocol in their products.
 """
+try:
+    from typing import Optional, Tuple
+except ImportError:
+    pass
 
 import struct
 
@@ -38,10 +42,10 @@ class _SettingsResult(ComplexCharacteristic):
 
     uuid = StandardUUID(0xFFF1)
 
-    def __init__(self):
+    def __init__(self) -> None:
         super().__init__(properties=Characteristic.NOTIFY)
 
-    def bind(self, service):
+    def bind(self, service: "IBBQService") -> _bleio.PacketBuffer:
         """Bind to an IBBQService."""
         bound_characteristic = super().bind(service)
         bound_characteristic.set_cccd(notify=True)
@@ -54,10 +58,10 @@ class _RealtimeData(ComplexCharacteristic):
 
     uuid = StandardUUID(0xFFF4)
 
-    def __init__(self):
+    def __init__(self) -> None:
         super().__init__(properties=Characteristic.NOTIFY)
 
-    def bind(self, service):
+    def bind(self, service: "IBBQService") -> _bleio.PacketBuffer:
         """Bind to an IBBQService."""
         bound_characteristic = super().bind(service)
         bound_characteristic.set_cccd(notify=True)
@@ -74,7 +78,7 @@ class IBBQService(Service):
     _UNITS_CELSIUS_MSG = b"\x02\x00\x00\x00\x00\x00"
     _REQUEST_BATTERY_LEVEL_MSG = b"\x08\x24\x00\x00\x00\x00"
 
-    def __init__(self, service=None):
+    def __init__(self, service: Optional["IBBQService"] = None) -> None:
         super().__init__(service=service)
         # Defer creating buffers until needed, since MTU is not known yet.
         self._settings_result_buf = None
@@ -106,27 +110,27 @@ def __init__(self, service=None):
     )
     """Send control messages here."""
 
-    def init(self):
+    def init(self) -> None:
         """Perform initial "pairing", which is not regular BLE pairing."""
         self.account_and_verify = self._CREDENTIALS_MSG
         self.settings_data = self._REALTIME_DATA_ENABLE_MSG
 
-    def display_fahrenheit(self):
+    def display_fahrenheit(self) -> None:
         """Display temperatures on device in degrees Fahrenheit.
 
         Note: This does not change the units returned by `temperatures`.
         """
         self.settings_data = self._UNITS_FAHRENHEIT_MSG
 
-    def display_celsius(self):
+    def display_celsius(self) -> None:
         """Display temperatures on device in degrees Celsius.
 
         Note: This does not change the units returned by `temperatures`.
         """
         self.settings_data = self._UNITS_CELSIUS_MSG
 
     @property
-    def temperatures(self):
+    def temperatures(self) -> Optional[Tuple]:
         """Return a tuple of temperatures for all the possible temperature probes on the device.
         Temperatures are in degrees Celsius. Unconnected probes return 0.0.
         """
@@ -145,7 +149,7 @@ def temperatures(self):
         return None
 
     @property
-    def battery_level(self):
+    def battery_level(self) -> Optional[Tuple[float, float]]:
         """Get current battery level in volts as ``(current_voltage, max_voltage)``.
         Results are approximate and may differ from the
         actual battery voltage by 0.1v or so.