Merge from master

Author: evaherrada

.github/workflows/build.yml

@@ -41,7 +41,7 @@ jobs:
       run: git describe --dirty --always --tags
     - name: PyLint
       run: |
-        pylint --disable=too-few-public-methods $( find . -path './adafruit*.py' )
+        pylint $( find . -path './adafruit*.py' )
         ([[ ! -d "examples" ]] || pylint --disable=missing-docstring,invalid-name,bad-whitespace examples/*.py)
     - name: Build assets
       run: circuitpython-build-bundles --filename_prefix ${{ steps.repo-name.outputs.repo-name }} --library_location .

adafruit_ble/advertising/__init__.py

@@ -85,6 +85,8 @@ def encode_data(data_dict, *, key_encoding="B"):
 
 class AdvertisingDataField:
     """Top level class for any descriptor classes that live in Advertisement or its subclasses."""
+    # pylint: disable=too-few-public-methods,unnecessary-pass
+    pass
 
 class AdvertisingFlag:
     """A single bit flag within an AdvertisingFlags object."""

adafruit_ble/attributes/__init__.py

@@ -63,6 +63,7 @@ class Attribute:
 
          security_mode: authenticated data signing, without man-in-the-middle protection
 """
+    # pylint: disable=too-few-public-methods
     NO_ACCESS = _bleio.Attribute.NO_ACCESS
     OPEN = _bleio.Attribute.OPEN
     ENCRYPT_NO_MITM = _bleio.Attribute.ENCRYPT_NO_MITM

adafruit_ble/characteristics/int.py

@@ -60,30 +60,36 @@ def __set__(self, obj, value):
 
 class Int8Characteristic(IntCharacteristic):
     """Int8 number."""
+    # pylint: disable=too-few-public-methods
     def __init__(self, *, min_value=-128, max_value=127, **kwargs):
         super().__init__("<b", min_value, max_value, **kwargs)
 
 class Uint8Characteristic(IntCharacteristic):
     """Uint8 number."""
+    # pylint: disable=too-few-public-methods
     def __init__(self, *, min_value=0, max_value=0xff, **kwargs):
         super().__init__("<B", min_value, max_value, **kwargs)
 
 class Int16Characteristic(IntCharacteristic):
     """Int16 number."""
+    # pylint: disable=too-few-public-methods
     def __init__(self, *, min_value=-32768, max_value=32767, **kwargs):
         super().__init__("<h", min_value, max_value, **kwargs)
 
 class Uint16Characteristic(IntCharacteristic):
     """Uint16 number."""
+    # pylint: disable=too-few-public-methods
     def __init__(self, *, min_value=0, max_value=0xffff, **kwargs):
         super().__init__("<H", min_value, max_value, **kwargs)
 
 class Int32Characteristic(IntCharacteristic):
     """Int32 number."""
+    # pylint: disable=too-few-public-methods
     def __init__(self, *, min_value=-2147483648, max_value=2147483647, **kwargs):
         super().__init__("<i", min_value, max_value, **kwargs)
 
 class Uint32Characteristic(IntCharacteristic):
     """Uint32 number."""
+    # pylint: disable=too-few-public-methods
     def __init__(self, *, min_value=0, max_value=0xffffffff, **kwargs):
         super().__init__("<I", min_value, max_value, **kwargs)

adafruit_ble/services/midi.py

@@ -36,6 +36,7 @@
 
 class MidiIOCharacteristic(Characteristic):
     """Workhorse MIDI Characteristic that carries midi messages both directions. Unimplemented."""
+    # pylint: disable=too-few-public-methods
     uuid = VendorUUID("7772E5DB-3868-4112-A1A9-F2669D106BF3")
     def __init__(self, **kwargs):
         super().__init__(properties=(Characteristic.NOTIFY |

adafruit_ble/services/standard/__init__.py

@@ -38,6 +38,7 @@
 
 class AppearanceCharacteristic(StructCharacteristic):
     """What type of device it is"""
+    # pylint: disable=too-few-public-methods
     uuid = StandardUUID(0x2a01)
 
     def __init__(self, **kwargs):

adafruit_ble/services/standard/hid.py

@@ -96,6 +96,7 @@ def send_report(self, report):
 
 class ReportOut:
     """A single HID report that receives HID data from a client."""
+    # pylint: disable=too-few-public-methods
     uuid = StandardUUID(_REPORT_UUID_NUM)
     def __init__(self, service, report_id, usage_page, usage, *, max_length):
         self._characteristic = _bleio.Characteristic.add_to_service(
@@ -270,7 +271,7 @@ def get_report_info(collection, reports):
             reports = {}
             get_report_info(collection, reports)
             if len(reports) > 1:
-                raise NotImplementedError("Only on report id per Application collection supported")
+                raise NotImplementedError("Only one report id per Application collection supported")
 
             report_id, report = list(reports.items())[0]
             output_size = report["output_size"]