-
Notifications
You must be signed in to change notification settings - Fork 94
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 |
@xirixiz ik ben ook al bezig met voorbereidingen te doen om DSNR-reader v6 werkend te krijgen in je project. Wellicht is het handiger dat we daar alles in meenemen. Die zit ook al op Python 3.13. Overigens zit het meeste werk in het overzetten en strippen van de DSMR-reader docs in DSMR-reader zelf, maar dat wordt wel wat makkelijker zodra de development-build van DSMR-reader 6 (vanuit |
Hehehe, ik kreeg geen notificaties van Github, alleen nadat je mij explicitet getagd hebt. Ik loop de comments nog even door 😄 |
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).