chore(ci): switch python_version to python-version

Bumps python from 3.11-slim to 3.13-slim.

---
updated-dependencies:
- dependency-name: python
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

.github/workflows/ci.yml

@@ -13,6 +13,8 @@ on:
 jobs:
   ci:
     uses: luketainton/gha-workflows/.github/workflows/ci-python-poetry-with-docker.yml@main
+    with:
+      python-version: 3.13
     secrets:
       SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
       SNYK_TOKEN: ${{ secrets.SNYK_TOKEN }}

.github/workflows/release.yml

@@ -5,6 +5,14 @@ on:
     - cron: "0 9 * * 0"
 
 jobs:
+  test:
+    uses: luketainton/gha-workflows/.github/workflows/ci-python-poetry-with-docker.yml@main
+    with:
+      python-version: 3.13
+    secrets:
+      SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
+      SNYK_TOKEN: ${{ secrets.SNYK_TOKEN }}
+
   create_release:
     name: Create Release
     uses: luketainton/gha-workflows/.github/workflows/create-release.yml@main

Dockerfile

@@ -1,4 +1,4 @@
-FROM python:3.11-slim
+FROM python:3.13-slim
 LABEL maintainer="Luke Tainton <luke@tainton.uk>"
 LABEL org.opencontainers.image.source="https://github.com/luketainton/roboluke-tasks"
 USER root
@@ -12,7 +12,9 @@ RUN mkdir -p /.local && \
 
 COPY pyproject.toml /run/pyproject.toml
 COPY poetry.lock /run/poetry.lock
-RUN poetry install --without dev --no-root
+
+RUN poetry config virtualenvs.create false && \
+    poetry install --without dev
 
 ENTRYPOINT ["python3", "-B", "-m", "app.main"]
 

app/utils/config.py

@@ -47,31 +47,27 @@ class Config:
         return os.environ["N8N_WEBHOOK_URL"]
 
     @property
-    def approved_users(self) -> list | None:
+    def approved_users(self) -> list:
         """Returns a list of approved users."""
         _emails: list[str] = os.environ.get("APPROVED_USERS", "").split(",")
         _emails: list[str] = [i.strip() for i in _emails if i]
         if not _emails:
-            return None
+            return []
         emails = [i for i in _emails if validate_email_syntax(i)]
         return emails
 
     @property
-    def approved_rooms(self) -> list | None:
+    def approved_rooms(self) -> list:
         """Returns a list of approved rooms."""
         _rooms: list[str] = os.environ.get("APPROVED_ROOMS", "").split(",")
         rooms: list[str] = [i.strip() for i in _rooms if i]
-        if not rooms:
-            return None
         return rooms
 
     @property
-    def approved_domains(self) -> list | None:
+    def approved_domains(self) -> list:
         """Returns a list of approved domains."""
         _domains: list[str] = os.environ.get("APPROVED_DOMAINS", "").split(",")
         domains: list[str] = [i.strip() for i in _domains if i]
-        if not domains:
-            return None
         return domains
 
 

tests/test_config_2.py

@@ -25,9 +25,9 @@ def test_config_no_admin_vars() -> None:
     # needs to be imported AFTER environment variables are set
     from app.utils.config import config  # pragma: no cover
 
-    assert config.approved_domains is None
-    assert config.approved_rooms is None
-    assert config.approved_users is None
+    assert config.approved_domains == []
+    assert config.approved_rooms == []
+    assert config.approved_users == []
     assert config.admin_emails == config_vars["ADMIN_EMAIL"].split(",")
     assert config.admin_first_name == config_vars["ADMIN_FIRST_NAME"]
     assert config.bot_name == config_vars["BOT_NAME"]