-
Notifications
You must be signed in to change notification settings - Fork 12
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 support for Django 4.X #35
Add support for Django 4.X #35
Conversation
Pull Request Test Coverage Report for Build 7532638881
💛 - Coveralls |
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, just had a few questions before approving
django-version: [ "3.2" , "4.0", "4.1", "4.2"] | ||
include: | ||
- python-version: "3.10" | ||
django-version: "5.0" |
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.
Any reason this is 5.0 vs one of the supported django-versions?
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.
Django 5.0 only supports Python 3.10 and above. That's why I added it as an include
@@ -1,4 +1,4 @@ | |||
# Core packages needed | |||
Django>=3.2, <4.0 | |||
Django>=3.2, <6.0 |
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.
Is there a reason this is 6.0 as opposed to the highest supported version?
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.
The highest version currently is 5.0. I want to make the package ready for the 5.X upgrade, so the next person that needs to update Django doesn't have any issues.
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.
LGTM
Overview
Add support for Django
4.0
,4.1
, and4.2
How to test