-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
Cleanup #9393
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
Cleanup #9393
Conversation
95b39d8
to
1468873
Compare
1468873
to
da15cad
Compare
8fb2a11
to
8030a12
Compare
This has gotten a little larger due to the last commit, but don't get scared, it's just whitespace changes ;-) Ready for review. |
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.
Wonderful, thanks.
but don't get scared, it's just whitespace changes
In general let's try to isolate changes as much as possible. The easier it is to review, the less likely we are to have sneaky errors creeping in. 😅
2c49951
to
9d4ed05
Compare
@tomchristie It looks like the docs changes from this PR have not been deployed on https://www.django-rest-framework.org/. How is that kicked off? |
Currently we push docs changes as part of releases. |
Maybe this should be included in release notes? |
Yes, probably. Sorry about that omission. Would you be able to create a PR? |
Sure: #9451 |
As discussed, this removes support for Django < 4.2 and Python < 3.8.
Further changes: