-
Notifications
You must be signed in to change notification settings - Fork 135
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
Add Django 5.0 and Python 3.12 to test matrix #204
Add Django 5.0 and Python 3.12 to test matrix #204
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #204 +/- ##
==========================================
- Coverage 98.95% 98.65% -0.30%
==========================================
Files 16 16
Lines 671 671
Branches 67 66 -1
==========================================
- Hits 664 662 -2
- Misses 4 5 +1
- Partials 3 4 +1 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, should tox.ini
be updated to include dj50
in the matrix as well?
tox.ini
Outdated
py{38,39,310,py311}-dj42 | ||
py{38,39,310,311}-dj41 | ||
py{38,39,310,311}-dj42 | ||
py{310,311,312}-dj42 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be py{310,311,312}-dj50
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤦 Yes, oops, fixed here: 96bae53
Yeah, let's bake a release after this is merged 👍 |
Awesome! Fixed the one thing you mentioned. I suppose I could merge it but probably best you do since it's my PR :) I'm also not familiar with how to get a release of this out, are there instructions? Or if you're able to just do it, that works too and thank you! |
The fix in #201 is required for Python 3.12 so wanted to add testing to cover that, and adding Django 5.0 support at the same time.
Should probably remove support for unsupported Django and Python versions, but that might require a breaking change version bump. Hoping we can get a release out soon that has the #201 change in it, and then remove support?