Skip to content

Clean up remaining code for dropped Django versions#407

Merged
codingjoe merged 4 commits intocodingjoe:masterfrom
KOliver94:drop-eol
Jun 9, 2025
Merged

Clean up remaining code for dropped Django versions#407
codingjoe merged 4 commits intocodingjoe:masterfrom
KOliver94:drop-eol

Conversation

@KOliver94
Copy link
Copy Markdown
Contributor

As #397 is conflicting I'm splitting it into 2 new PRs. In this one I'm removing all no more officially supported Django versions.

@KOliver94 KOliver94 mentioned this pull request Jan 28, 2024
@seanpar203
Copy link
Copy Markdown
Collaborator

Going to talk with Frank about this and then i'll circle back!

@KOliver94 KOliver94 force-pushed the drop-eol branch 2 times, most recently from 57dd2c5 to ca89b41 Compare June 8, 2025 22:43
@KOliver94
Copy link
Copy Markdown
Contributor Author

Hi @codingjoe,

Although #463 removed support for EOL Django and Python versions, there are still several parts of the codebase that include compatibility for them. I've updated my earlier PR to fully drop support for Django versions older than 4.2 as well.

@KOliver94 KOliver94 changed the title Drop support for older Django version Clean up remaining code for dropped Django versions Jun 8, 2025
@codingjoe codingjoe self-requested a review June 9, 2025 16:33
Copy link
Copy Markdown
Owner

@codingjoe codingjoe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @KOliver94,

Excellent! I appreciate the support. I wanted to get the packaging update out of the door without any changes to the code. With that out of the way, it's high time to drop the legacy code.

I added a commit where I used django-upgrade to spot a few other legacy lines.

Thanks again!
Joe

@codingjoe codingjoe merged commit d516616 into codingjoe:master Jun 9, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants