Out with setup, in with poetry

Migrated the package from setuptools to using pypackage and poetry.
Also moved all (most) of the scripts from the contrib directory
into blynclight/scripts package and gave them all click interfaces.
Added the magic to pypackage to enable installing the scripts as
commands. Which is nice.

Author: JnyJny

.pre-commit-config.yaml

@@ -0,0 +1,14 @@
+# See https://pre-commit.com for more information
+# See https://pre-commit.com/hooks.html for more hooks
+repos:
+-   repo: https://github.com/pre-commit/pre-commit-hooks
+    rev: v2.2.1
+    hooks:
+    -   id: trailing-whitespace
+    -   id: end-of-file-fixer
+    -   id: check-yaml
+    -   id: check-added-large-files
+-   repo: https://github.com/ambv/black
+    rev: stable
+    hooks:
+    -   id: black

VERSION

@@ -35,11 +35,7 @@
 
 from .constants import FlashSpeed, MusicSelections, MusicVolume
 
-from .exceptions import (
-    BlyncLightNotFound,
-    BlyncLightUnknownDevice,
-    BlyncLightInUse,
-)
+from .exceptions import BlyncLightNotFound, BlyncLightUnknownDevice, BlyncLightInUse
 
 from .proxy import Proxy as BlyncLightProxy
 

src/blynclight/__init__.py blynclight/__init__.pysrc/blynclight/__init__.py renamed to blynclight/__init__.py

@@ -3,17 +3,8 @@
 from ctypes import Structure, c_uint64
 from contextlib import contextmanager
 from .hid import HidDevice
-from .constants import (
-    EMBRAVA_VENDOR_IDS,
-    END_OF_COMMAND,
-    COMMAND_LENGTH,
-    PAD_VALUE,
-)
-from .exceptions import (
-    BlyncLightNotFound,
-    BlyncLightUnknownDevice,
-    BlyncLightInUse,
-)
+from .constants import EMBRAVA_VENDOR_IDS, END_OF_COMMAND, COMMAND_LENGTH, PAD_VALUE
+from .exceptions import BlyncLightNotFound, BlyncLightUnknownDevice, BlyncLightInUse
 
 
 class BlyncLight(Structure):
@@ -52,10 +43,10 @@ class method get_light().
 
     Additionally, any updates to the bit fields in the ByncLight class
     will be immediately written to the hardware device by default.
-    
+
     Finally, when a BlyncLight object is deallocated it will attempt
     to return the light to a known quiescent state before releasing
-    the device. 
+    the device.
 
     Callers can defer hardware updates by setting the 'immediate'
     attribute to False. Any changes to command fields will not be
@@ -103,10 +94,10 @@ class method get_light().
     green  : 8     Green component varies between 0-255
     off    : 1     0==on     1==off
     dim    : 1     0==bright 1==dim
-    flash  : 1     0==steady 1==flash  
+    flash  : 1     0==steady 1==flash
     speed  : 3     0==off 1==low 2==medium 4==fast
     pad    : 2
-    mute   : 1     0==unmute 1==mute 
+    mute   : 1     0==unmute 1==mute
     music  : 4     select a built-in musical tune
     play   : 1     0==stop 1==play
     repeat : 1     0==no repeat 1==repeat
@@ -235,23 +226,21 @@ def __init__(self, vendor_id, product_id, immediate=True):
 
         If immediate is False, the caller will need to set immediate
         to True before updates to the device will occur.
-        
+
         The following exceptions are raised:
 
         - BlyncLightUnknownDevice is raised if vendor_id does not
           match known Embrava vendor indentifiers.
-        
+
         - BlyncLightInUse is raised if the light specified by
           vendor_id:product_id has already been opened.
-        
+
         - BlyncLightNotFound is raised if the light specified by
           vendor_id:product_id does not exist.
 
         """
         if vendor_id not in EMBRAVA_VENDOR_IDS:
-            raise BlyncLightUnknownDevice(
-                f"unknown vendor id 0x{vendor_id:04}"
-            )
+            raise BlyncLightUnknownDevice(f"unknown vendor id 0x{vendor_id:04}")
         self.vendor_id = vendor_id
         self.product_id = product_id
         self.reset(flush=False)
@@ -325,9 +314,7 @@ def __setattr__(self, name, value):
         if name in self.commands and self.immediate:
             n = self.device.write(self.bytes)
             if n != len(self.bytes):
-                raise IOError(
-                    f"wrote {n} bytes, expected {len(self.bytes)} bytes"
-                )
+                raise IOError(f"wrote {n} bytes, expected {len(self.bytes)} bytes")
 
     @property
     def device(self):

src/blynclight/blynclight.py blynclight/blynclight.pysrc/blynclight/blynclight.py renamed to blynclight/blynclight.py

@@ -88,12 +88,7 @@ class _DeviceHandle(ctypes.Structure):
     pass
 
 
-_hidapi_libraries = [
-    "hidapi",
-    "hidapi-hidraw",
-    "hidapi-libusb",
-    "hidapi-iohidmanager",
-]
+_hidapi_libraries = ["hidapi", "hidapi-hidraw", "hidapi-libusb", "hidapi-iohidmanager"]
 
 for stem in _hidapi_libraries:
     fullname = ctypes.util.find_library(stem)
@@ -104,11 +99,7 @@ class _DeviceHandle(ctypes.Structure):
     raise ImportError("failed to locate hidapi shared object")
 
 _function_signatures = [
-    (
-        "hid_enumerate",
-        [ctypes.c_ushort, ctypes.c_ushort],
-        ctypes.POINTER(_DeviceInfo),
-    ),
+    ("hid_enumerate", [ctypes.c_ushort, ctypes.c_ushort], ctypes.POINTER(_DeviceInfo)),
     ("hid_free_enumeration", [ctypes.POINTER(_DeviceInfo)], None),
     (
         "hid_open",
@@ -119,20 +110,12 @@ class _DeviceHandle(ctypes.Structure):
     ("hid_close", [ctypes.POINTER(_DeviceHandle)], None),
     (
         "hid_write",
-        [
-            ctypes.POINTER(_DeviceHandle),
-            ctypes.POINTER(ctypes.c_byte),
-            ctypes.c_size_t,
-        ],
+        [ctypes.POINTER(_DeviceHandle), ctypes.POINTER(ctypes.c_byte), ctypes.c_size_t],
         ctypes.c_int,
     ),
     (
         "hid_read",
-        [
-            ctypes.POINTER(_DeviceHandle),
-            ctypes.POINTER(ctypes.c_byte),
-            ctypes.c_size_t,
-        ],
+        [ctypes.POINTER(_DeviceHandle), ctypes.POINTER(ctypes.c_byte), ctypes.c_size_t],
         ctypes.c_int,
     ),
     (
@@ -158,7 +141,7 @@ class _DeviceHandle(ctypes.Structure):
 
 class HidDevice:
     """A USB Human Input Device (HID) Device
-    
+
     Provides read and write access to devices that adhere to the USB
     HID protocol. The HidDevice.enumerate class method can be used to
     discover currently attached devices.
@@ -256,9 +239,7 @@ def handle(self):
             return self._handle
         except AttributeError:
             pass
-        self._handle = hidapi.hid_open(
-            self.vendor_id, self.product_id, self.path
-        )
+        self._handle = hidapi.hid_open(self.vendor_id, self.product_id, self.path)
         if not self._handle:
             raise LookupError(f"no such device: {self.identifier}")
         self._opened.add(self.identifier)

src/blynclight/constants.py blynclight/constants.pysrc/blynclight/constants.py renamed to blynclight/constants.py

@@ -62,7 +62,7 @@ def color(self, light_id, value):
         :param value: color specifier ( tuple of ints or int )
 
         Changes the color of the light associated with light_id.
-        
+
         """
         if isinstance(value, tuple):
             # convert tuple to string

src/blynclight/exceptions.py blynclight/exceptions.pysrc/blynclight/exceptions.py renamed to blynclight/exceptions.py

@@ -16,35 +16,17 @@
 @click.option("--flash/--no-flash", default=False)
 @click.option("-s", "--speed", default=0, help="0, 1, 2, 4")
 @click.option(
-    "-v",
-    "--verbose",
-    is_flag=True,
-    default=False,
-    help="Prints the light status.",
+    "-v", "--verbose", is_flag=True, default=False, help="Prints the light status."
 )
 @click.option("-d", "--duration", default=-1)
 @click.option(
-    "-a",
-    "--available",
-    is_flag=True,
-    default=False,
-    help="Show available lights.",
+    "-a", "--available", is_flag=True, default=False, help="Show available lights."
 )
 def cli(
-    light_id,
-    red,
-    blue,
-    green,
-    off,
-    bright,
-    flash,
-    speed,
-    verbose,
-    duration,
-    available,
+    light_id, red, blue, green, off, bright, flash, speed, verbose, duration, available
 ):
     """Initialize the state of a connected BlyncLight and
-    activate the light for 'duration' seconds. 
+    activate the light for 'duration' seconds.
     """
 
     if available: