pylint caught an inconsistency

Author: dhalbert

adafruit_ble/advertising/__init__.py

@@ -86,7 +86,7 @@ def encode_data(data_dict, *, key_encoding="B"):
         struct.pack_into(key_encoding, data, i + 1, key)
         data[i + 1 + key_size : i + 1 + item_length] = bytes(value)
         i += 1 + item_length
-    return data
+    return bytes(data)
 
 
 class AdvertisingDataField:

adafruit_ble/characteristics/__init__.py

@@ -51,7 +51,8 @@ class Characteristic:
     :param int write_perm: Specifies whether the characteristic can be written by a client,
        and if so, which security mode is required. Values allowed are the same as ``read_perm``.
     :param int max_length: Maximum length in bytes of the characteristic value. The maximum allowed
-       is 512, or on nRF, 510 if ``fixed_length`` is True. The default, 20, is the maximum
+       by the BLE specification is 512. On nRF, if ``fixed_length`` is ``True``, the maximum
+       is 510. The default value is 20, which is the maximum
        number of data bytes that fit in a single BLE 4.x ATT packet.
     :param bool fixed_length: True if the characteristic value is of fixed length.
     :param buf initial_value: The initial value for this characteristic. If not given, will be