Skip to content

chore: drop unused python-dateutil, test on Python 3.14, mypy targets… - #574

Open
cdnninja wants to merge 1 commit into
webdjoe:devfrom
cdnninja:claude/cleanup
Open

cdnninja wants to merge 1 commit into
webdjoe:devfrom
cdnninja:claude/cleanup

Conversation

@cdnninja

Copy link
Copy Markdown
Contributor

… 3.11

  • requirements.txt listed python-dateutil, but nothing imports it and pyproject.toml never declared it.
  • Add Python 3.14 to the CI matrix and tox envlist, and update the docs that list the tested versions.
  • mypy.ini targeted 3.12 while requires-python is >=3.11, so mypy could accept syntax or stdlib APIs the minimum version lacks.

Claude-Session: https://claude.ai/code/session_01CxGYP1MF5XktdJeS1hyNaf

Description

Type of change

  • Bug fix (fix:)
  • New feature or device support (feat:)
  • Documentation (docs:)
  • Refactor / test / chore (no functional change)

Checklist

  • This PR is a single feature, fix, or edit (not a bundle of unrelated changes).
  • The change follows the intended architecture (base class → device + API mixin → device map; state in DeviceState; models in models/; constants in pyvesync.const).
  • No API request/response fields were invented — all come from a real packet capture.
  • ruff check, pylint, mypy, and pytest pass locally.
  • Tests added/updated and API fixtures written (pytest --write_api) where relevant.
  • Documentation updated if behavior or public API changed.

Real-hardware verification

  • Tested on the physical device.
    • Model: ____
    • Firmware version: ____
  • Not applicable (explain why): ____
  • I do not own the device; a packet capture is provided / the device has been shared with a maintainer for verification.

… 3.11

- requirements.txt listed python-dateutil, but nothing imports it and
  pyproject.toml never declared it.
- Add Python 3.14 to the CI matrix and tox envlist, and update the docs
  that list the tested versions.
- mypy.ini targeted 3.12 while requires-python is >=3.11, so mypy could
  accept syntax or stdlib APIs the minimum version lacks.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CxGYP1MF5XktdJeS1hyNaf

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant