-
-
Notifications
You must be signed in to change notification settings - Fork 486
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
Updated test matrix to test against Django 5.2 explicitly #1441
Conversation
Co-authored-by: Anders <[email protected]>
I committed the suggested changes. I included the alpha mention because it may not be clear to others and it doesn't hurt anything to be explicit. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1441 +/- ##
==========================================
+ Coverage 96.87% 97.02% +0.15%
==========================================
Files 24 24
Lines 1470 1479 +9
Branches 240 189 -51
==========================================
+ Hits 1424 1435 +11
Misses 25 25
+ Partials 21 19 -2 ☔ 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.
Nice :)
Description
Updated test matrix to use Django 5.2
Related Issue
Motivation and Context
Test coverage and the main ci action was broken.
How Has This Been Tested?
It hasn't yet. We'll see how ci does.
Screenshots (if appropriate):
Types of changes
Checklist:
pre-commit run
command to format and lint.AUTHORS.rst
CHANGES.rst