-
-
Notifications
You must be signed in to change notification settings - Fork 498
Add support for Django 5.2 #786
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
Conversation
This confirms support for Django 5.2 by modifying the test matrices, classifiers, and documentation alongside currently supported versions 4.2, 5.0, and 5.1.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #786 +/- ##
==========================================
- Coverage 74.07% 69.74% -4.34%
==========================================
Files 26 26
Lines 1678 1679 +1
==========================================
- Hits 1243 1171 -72
- Misses 435 508 +73 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
dulmandakh
left a comment
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.
Please fix the CI
|
I guess we want to first find out why #781 fails. |
|
Hey, thanks all! I'm not super-familiar with the individual test that is failing in the Any guidance you can provide? I also can't reproduce this on my local (Mac OS 15.6.1 w/ ImageMagick v7.x, versus the v6.9.12.98 which seems to be being run in Ubuntu 24.04 in Github Actions). I'm working on trying to reproduce via a Docker image, but that is not going quickly. |
|
I just realized neither of approaches to fixing the CI failures is likely to work as described in the commit message of I would definitely welcome feedback here on how to handle this. |
|
@dulmandakh, based on my commit message and comment above, as well as @claudep's point about #781, I think we can all agree this is a pre-existing issue to this PR. Looking at the CI status for the second commit (which does fix some CI issues), you can see that only the ImageMagick related tests are failing. I'd be happy to remove my follow up attempt commit to fix the remaining CI issues and create a new bug issue for it. |
|
Please just remove stuff not Django 5.2-related, and we'll push even if tests fail, as these are unrelated issues. |
c46636d to
c407dbf
Compare
|
@claudep, apologies for missing the notification. I've removed the other commits and force pushed. |
This confirms support for Django 5.2 by modifying the test matrices, classifiers, and documentation alongside currently supported versions 4.2, 5.0, and 5.1.
I did not remove Django 5.0 support yet, even though it stopped being supported on April 2, 2025. I would be happy to add a commit to do so.