-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
Update test matrix to use Django 5.2 stable version #9679
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
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.
Sure, small clean-up. Although this technically doesn't change anything (tox started to pick up the Django 5.2 final as soon as it got released).
Ah, we specified a minimum version, so I guess that's how it works :) Thanks for reviewing the PR! |
There is some satisfaction of having all versions consistent... We could switch to |
Not sure about the benefit with |
Less churn: we update the matrix once when first adding support and don't bother with following up with other PRs when the final version is out |
Yeah, I think it's more of a coding convention than a technical issue, and once you've chosen that convention, I think we'll have fewer problems writing or reviewing code like this in the future. Ideally, the DRF should be released along with the major release of Django, so we could choose the following workflow:
|
Thanks for reviewing my PR, all! :) |
Description
Now that django 5.2 is generally available, update the tox test from using a beta version of Django 5.2 to the general release.