Add Python 3.12, Drop Python 3.10 (#1134)

Author: frenck

.github/labels.yml

@@ -4,7 +4,7 @@
   description: "A breaking change for existing users."
 - name: "bugfix"
   color: ee0701
-  description: "Inconsistencies or issues which will cause a problem for users or implementors."
+  description: "Inconsistencies or issues which will cause a problem for users or implementers."
 - name: "documentation"
   color: 0052cc
   description: "Solely about the documentation of the project."

.github/workflows/linting.yaml

@@ -8,7 +8,7 @@ on:
   workflow_dispatch:
 
 env:
-  DEFAULT_PYTHON: "3.10"
+  DEFAULT_PYTHON: "3.11"
 
 jobs:
   codespell:

.github/workflows/release.yaml

@@ -8,7 +8,7 @@ on:
       - published
 
 env:
-  DEFAULT_PYTHON: "3.10"
+  DEFAULT_PYTHON: "3.11"
 
 jobs:
   release:

.github/workflows/tests.yaml

@@ -8,15 +8,15 @@ on:
   workflow_dispatch:
 
 env:
-  DEFAULT_PYTHON: "3.10"
+  DEFAULT_PYTHON: "3.11"
 
 jobs:
   pytest:
     name: Python ${{ matrix.python }}
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python: ["3.10", "3.11"]
+        python: ["3.11"]
     steps:
       - name: ⤵️ Check out code from GitHub
         uses: actions/checkout@v4.1.1

.github/workflows/typing.yaml

@@ -8,7 +8,7 @@ on:
   workflow_dispatch:
 
 env:
-  DEFAULT_PYTHON: "3.10"
+  DEFAULT_PYTHON: "3.11"
 
 jobs:
   mypy:

README.md

@@ -87,7 +87,7 @@ development.
 
 You need at least:
 
-- Python 3.10+
+- Python 3.11+
 - [Poetry][poetry-install]
 - NodeJS 18+ (including NPM)
 

pyproject.toml

@@ -5,8 +5,8 @@ classifiers = [
   "Framework :: AsyncIO",
   "Intended Audience :: Developers",
   "Natural Language :: English",
-  "Programming Language :: Python :: 3.10",
   "Programming Language :: Python :: 3.11",
+  "Programming Language :: Python :: 3.12",
   "Programming Language :: Python :: 3",
   "Topic :: Software Development :: Libraries :: Python Modules",
 ]
@@ -29,7 +29,7 @@ aiohttp = ">=3.0.0"
 awesomeversion = ">=22.1.0"
 backoff = ">=2.2.0"
 cachetools = ">=4.0.0"
-python = "^3.10"
+python = "^3.11"
 yarl = ">=1.6.0"
 
 [tool.poetry.urls]
@@ -68,7 +68,7 @@ show_missing = true
 # free to run mypy on Windows, Linux, or macOS and get consistent
 # results.
 platform = "linux"
-python_version = "3.10"
+python_version = "3.11"
 
 # show error messages from unrelated files
 follow_imports = "normal"

sonar-project.properties

@@ -13,5 +13,5 @@ sonar.sourceEncoding=UTF-8
 sonar.sources=src
 sonar.tests=tests
 
-sonar.python.version=3.10, 3.11
+sonar.python.version=3.11, 3.12
 sonar.python.coverage.reportPaths=coverage.xml

src/wled/models.py

@@ -130,6 +130,7 @@ class Segment:
     stop: int
 
     @staticmethod
+    # pylint: disable-next=too-many-arguments
     def from_dict(  # noqa: PLR0913
         segment_id: int,
         data: dict[str, Any],
@@ -665,11 +666,11 @@ def from_dict(
 class Device:
     """Object holding all information of WLED."""
 
-    effects: list[Effect] = []
+    effects: list[Effect]
     info: Info
-    palettes: list[Palette] = []
-    playlists: list[Playlist] = []
-    presets: list[Preset] = []
+    palettes: list[Palette]
+    playlists: list[Playlist]
+    presets: list[Preset]
     state: State
 
     def __init__(self, data: dict[str, Any]) -> None:
@@ -684,6 +685,11 @@ def __init__(self, data: dict[str, Any]) -> None:
             WLEDError: In case the given API response is incomplete in a way
                 that a Device object cannot be constructed from it.
         """
+        self.effects = []
+        self.palettes = []
+        self.playlists = []
+        self.presets = []
+
         # Check if all elements are in the passed dict, else raise an Error
         if any(
             k not in data and data[k] is not None

src/wled/wled.py

@@ -6,7 +6,7 @@
 import socket
 from contextlib import suppress
 from dataclasses import dataclass
-from typing import TYPE_CHECKING, Any
+from typing import TYPE_CHECKING, Any, Self
 
 import aiohttp
 import async_timeout
@@ -198,11 +198,11 @@ async def request(
                 response.close()
 
                 if content_type == "application/json":
-                    raise WLEDError(  # noqa: TRY301
+                    raise WLEDError(
                         response.status,
                         json.loads(contents.decode("utf8")),
                     )
-                raise WLEDError(  # noqa: TRY301
+                raise WLEDError(
                     response.status,
                     {"message": contents.decode("utf8")},
                 )
@@ -367,7 +367,7 @@ async def master(
 
         await self.request("/json/state", method="POST", data=state)
 
-    # pylint: disable=too-many-locals, too-many-branches
+    # pylint: disable=too-many-locals, too-many-branches, too-many-arguments
     async def segment(  # noqa: PLR0912, PLR0913
         self,
         segment_id: int,
@@ -689,14 +689,15 @@ async def upgrade(self, *, version: str | AwesomeVersion) -> None:
         )
 
         try:
-            async with async_timeout.timeout(self.request_timeout * 10):
-                async with self.session.get(
-                    download_url,
-                    raise_for_status=True,
-                ) as download:
-                    form = aiohttp.FormData()
-                    form.add_field("file", await download.read(), filename=update_file)
-                    await self.session.post(url, data=form)
+            async with async_timeout.timeout(
+                self.request_timeout * 10,
+            ), self.session.get(
+                download_url,
+                raise_for_status=True,
+            ) as download:
+                form = aiohttp.FormData()
+                form.add_field("file", await download.read(), filename=update_file)
+                await self.session.post(url, data=form)
         except asyncio.TimeoutError as exception:
             msg = "Timeout occurred while fetching WLED version information from GitHub"
             raise WLEDConnectionTimeoutError(msg) from exception
@@ -804,7 +805,7 @@ async def close(self) -> None:
         if self.session and self._close_session:
             await self.session.close()
 
-    async def __aenter__(self) -> WLED:
+    async def __aenter__(self) -> Self:
         """Async enter.
 
         Returns
@@ -813,7 +814,7 @@ async def __aenter__(self) -> WLED:
         """
         return self
 
-    async def __aexit__(self, *_exc_info: Any) -> None:
+    async def __aexit__(self, *_exc_info: object) -> None:
         """Async exit.
 
         Args: