Skip to content
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

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

xirixiz
Copy link
Contributor

@xirixiz xirixiz commented Dec 30, 2024

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 main v*-branch(es).

Copy link

codecov bot commented Dec 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.61%. Comparing base (9ee2d88) to head (20cf625).
Report is 121 commits behind head on development.

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           

@dennissiemensma
Copy link
Member

dennissiemensma commented Dec 30, 2024

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

@dennissiemensma
Copy link
Member

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.

@dennissiemensma
Copy link
Member

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.

@dennissiemensma
Copy link
Member

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

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.

2 participants