Update test matrix to use Django 5.2 stable version#9679
Update test matrix to use Django 5.2 stable version#9679browniebroke merged 2 commits intoencode:masterfrom
Conversation
browniebroke
left a comment
There was a problem hiding this comment.
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.