Typed

.github/workflows/publish.yml

@@ -0,0 +1,55 @@
+name: Publish to PyPI
+
+on:
+  push:
+    tags:
+      - "v*"
+
+permissions:
+  contents: read
+
+jobs:
+  build:
+    name: Build distribution
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.12"
+      - name: Verify tag matches package version
+        run: |
+          version="$(python setup.py --version)"
+          tag="${GITHUB_REF_NAME#v}"
+          if [ "$version" != "$tag" ]; then
+            echo "Tag $tag does not match package version $version" >&2
+            exit 1
+          fi
+      - name: Build sdist and wheel
+        run: |
+          python -m pip install --upgrade pip build
+          python -m build
+      - name: Upload distribution artifacts
+        uses: actions/upload-artifact@v4
+        with:
+          name: dist
+          path: dist/
+
+  publish:
+    name: Publish to PyPI
+    needs: build
+    runs-on: ubuntu-latest
+    environment:
+      name: pypi
+      url: https://pypi.org/project/python-dali/
+    permissions:
+      id-token: write
+    steps:
+      - name: Download distribution artifacts
+        uses: actions/download-artifact@v4
+        with:
+          name: dist
+          path: dist/
+      - name: Publish to PyPI
+        uses: pypa/gh-action-pypi-publish@release/v1

.github/workflows/run-tests.yml

@@ -8,7 +8,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ['3.8', '3.9', '3.10', '3.11', '3.12']
+        python-version: ['3.10', '3.11', '3.12']
 
     steps:
       - uses: actions/checkout@v3

MANIFEST.in

@@ -0,0 +1 @@
+recursive-include dali py.typed

dali/address.py

@@ -20,6 +20,8 @@
 7.2.2.  Decoding of event messages is currently not implemented.
 """
 
+from __future__ import annotations
+
 from dali.exceptions import IncompatibleFrame
 
 
@@ -261,7 +263,7 @@ class GearShort(GearAddress):
     address.
     """
 
-    def __init__(self, address):
+    def __init__(self, address: int) -> None:
         if not isinstance(address, int):
             raise ValueError("address must be an integer")
         if address < 0 or address > 63:
@@ -305,7 +307,7 @@ class DeviceShort(DeviceAddress):
     address.
     """
 
-    def __init__(self, address: int):
+    def __init__(self, address: int) -> None:
         if not isinstance(address, int):
             raise ValueError("address must be an integer")
         if address < 0 or address > 63:
@@ -379,7 +381,7 @@ def __str__(self):
 class _AddressedInstance(Instance):
     _flags = None
 
-    def __init__(self, value):
+    def __init__(self, value: int) -> None:
         if not isinstance(value, int):
             raise ValueError("value must be an integer")
         if value < 0 or value > 31:

dali/command.py

@@ -1,6 +1,9 @@
 """Declaration of base types for dali commands and their responses."""
 
+from __future__ import annotations
+
 from enum import IntEnum
+from typing import Any
 from dali import address
 from dali import frame
 from dali.exceptions import MissingResponse
@@ -46,17 +49,17 @@ class Response:
     _expected = False
     _error_acceptable = False
 
-    def __init__(self, val):
+    def __init__(self, val: frame.BackwardFrame | None) -> None:
         if val is not None and not isinstance(val, frame.BackwardFrame):
             raise TypeError("Response must be passed None or a BackwardFrame")
         self._value = val
 
     @property
-    def raw_value(self):
+    def raw_value(self) -> frame.BackwardFrame | None:
         return self._value
 
     @property
-    def value(self):
+    def value(self) -> Any:
         if self._value is None and self._expected:
             raise MissingResponse()
         if self._value and self._value.error and not self._error_acceptable:

dali/device/__init__.py

@@ -9,3 +9,4 @@
 import dali.device.pushbutton  # noqa: F401
 import dali.device.occupancy  # noqa: F401
 import dali.device.light  # noqa: F401
+import dali.device.general_sensor  # noqa: F401

dali/device/general.py

@@ -128,7 +128,7 @@ class _StandardDeviceCommand(_DeviceCommand):
     """
     _opcode = None
 
-    def __init__(self, device):
+    def __init__(self, device: address.Address | int) -> None:
         if self._opcode is None:
             raise NotImplementedError
 
@@ -514,7 +514,9 @@ class _StandardInstanceCommand(_DeviceCommand):
 
     _opcode = None
 
-    def __init__(self, device, instance):
+    def __init__(
+        self, device: address.Address | int, instance: address.Instance
+    ) -> None:
         if self._opcode is None:
             raise NotImplementedError
 
@@ -784,7 +786,7 @@ def __str__(self):
 
 
 class _SpecialDeviceCommandOneParam(_SpecialDeviceCommand):
-    def __init__(self, param):
+    def __init__(self, param: int) -> None:
         if not isinstance(param, int):
             raise ValueError("parameter must be an integer")
         if param < 0 or param > 255: