-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support Python 3.13 (psycopg2 package) #2044
base: development
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## development #2044 +/- ##
============================================
Coverage 99.60% 99.61%
============================================
Files 129 129
Lines 4597 4618 +21
Branches 443 455 +12
============================================
+ Hits 4579 4600 +21
Misses 17 17
Partials 1 1 |
Ik weet niet of dit met de huidige versie van Django (3.2) in DSMR-reader werkt. Officieel ondersteunen ze maar tot Python 3.10: https://docs.djangoproject.com/en/5.1/releases/3.2/#python-compatibility Je kunt je PR nog uitbreiden door hier Python 3.13 van te maken en te kijken of dat werkt: https://github.com/dsmrreader/dsmr-reader/blob/development/.github/workflows/automated-tests.yml#L21 En aan dit lijstje nog 3.12 toe te voegen: https://github.com/dsmrreader/dsmr-reader/blob/development/.github/workflows/automated-tests.yml#L125 |
Van de zomer ben ik al bezig geweest om dit permanent op te lossen, door je Docker-setup voor DSMR-reader 6+ als enige installatie nog te ondersteunen. Waardoor ik nog maar 1 versie hoef te ondersteunen zonder gedoe.
|
Helaas heb ik eind van de zomer ernstig enkeleletsel gekregen na een ongeluk, dus mijn plannen voor ontwikkelen voor de winter liggen nog even stil (welliswaar weinig pijn bij achter computer zitten, maar lang zitten zorgt niet voor herstel) Ik ben wel een heel eind met de bovenstaande setup. Ik was bezig om alle docs te versimpelen (weggooien wat niet meer relevant is voor Docker) en daarna nog een upgrade guide te maken. Dus daar hangt het momenteel op. Wellicht dat ik er binnenkort weer meer tijd voor heb, gezien ik (tijdelijk) overweeg een dagje minder te gaan werken. Al moet dat het herstel niet in de weg zitten. |
Voor DSMR-reader 6 lijkt 3.13 al wel te werken. Ik zal 3.12 voor de zekerheid ook nog laten testen in de CI, maar over een paar maanden zal 3.13 de mainstream versie worden. https://github.com/dsmrreader/dsmr-reader/actions/runs/12656617088 |
Support for python 3.13
Package psycopg2
https://github.com/psycopg/psycopg2/blob/master/NEWS
Important
Always create your pull request from the DSMR-reader
development
-branch and not the mainv*
-branch(es).