chore: drop python 3.7 support (#284)

.github/workflows/ci.yml

@@ -36,7 +36,6 @@ jobs:
       fail-fast: false
       matrix:
         python-version:
-          - "3.7"
           - "3.8"
           - "3.9"
           - "3.10"
@@ -135,13 +134,13 @@ jobs:
           fetch-depth: 0
 
       - name: Install cibuildwheel
-        run: python -m pip install cibuildwheel==2.16.2
+        run: python -m pip install cibuildwheel==2.19.1
 
       - name: Build wheels
         run: python -m cibuildwheel --output-dir wheelhouse
         # to supply options, put them in 'env', like:
         env:
-          CIBW_SKIP: cp36-*
+          CIBW_SKIP: cp36-* cp37-*
           CIBW_BEFORE_ALL_LINUX: apt-get install -y gcc || yum install -y gcc || apk add gcc
           CIBW_BUILD_VERBOSITY: 3
           REQUIRE_CYTHON: 1

pyproject.toml

@@ -27,7 +27,7 @@ script = "build_ext.py"
 "Changelog" = "https://github.com/bluetooth-devices/dbus-fast/blob/main/CHANGELOG.md"
 
 [tool.poetry.dependencies]
-python = "^3.7"
+python = "^3.8"
 
 # duplicated in docs/requirements.txt for readthedocs compatibility
 [tool.poetry.group.docs.dependencies]

src/dbus_fast/_private/marshaller.py

@@ -1,5 +1,5 @@
 from struct import Struct, error
-from typing import Any, Callable, Dict, Iterable, List, Optional, Tuple, Union
+from typing import Any, Callable, Dict, List, Optional, Tuple, Union
 
 from ..signature import SignatureType, Variant, get_signature_tree
 

src/dbus_fast/_private/unmarshaller.py

@@ -513,10 +513,8 @@ class Unmarshaller:
             -self._pos & (UINT32_SIZE - 1)
         ) + UINT32_SIZE  # align for the uint32
         if self._is_native and cython.compiled:
-            array_length = (
+            array_length = _cast_uint32_native(  # type: ignore[name-defined] # pragma: no cover
-                _cast_uint32_native(  # type: ignore[name-defined] # pragma: no cover
+                self._buf, self._pos - UINT32_SIZE
-                    self._buf, self._pos - UINT32_SIZE
-                )
             )
         else:
             array_length = self._uint32_unpack(self._buf, self._pos - UINT32_SIZE)[0]  # type: ignore[misc]

src/dbus_fast/aio/message_reader.py

@@ -21,7 +21,7 @@ def _message_reader(
                 return
             try:
                 process(message)
-            except Exception as e:
+            except Exception:
                 logging.error("Unexpected error processing message: %s", exc_info=True)
             # If we are not negotiating unix fds, we can stop reading as soon as we have
             # the buffer is empty as asyncio will call us again when there is more data.

tests/test_auth.py

@@ -1,5 +1,4 @@
 """This tests setting a hardcoded UID in AuthExternal"""
-import os
 
 import pytest
 

tests/test_fd_passing.py

@@ -1,4 +1,5 @@
 """This tests the ability to send and receive file descriptors in dbus messages"""
+
 import os
 
 import pytest

tests/test_unpack_variants.py

@@ -1,4 +1,5 @@
 """Test unpack variants."""
+
 import pytest
 
 from dbus_fast.signature import Variant