Skip to content

Conversation

@edmorley
Copy link
Contributor

@edmorley edmorley commented Jul 1, 2025

Since it was only being used for TypedDict, which was added in Python 3.8 and dj-database-url v2.3.0+ only supports Python 3.9+.

See:
https://docs.python.org/3.12/library/typing.html#typing.TypedDict
https://github.com/jazzband/dj-database-url/blob/master/CHANGELOG.md#v230-2024-10-23

Note: This is a repeat of #257, since that PR was lost as part of merges that occurred during the release of 3.0.0:
#257 (comment)

Since it was only being used for `TypedDict`, which was added in
Python 3.8 and `dj-database-url` v2.3.0+ only supports Python 3.9+.

See:
https://docs.python.org/3.12/library/typing.html#typing.TypedDict
https://github.com/jazzband/dj-database-url/blob/master/CHANGELOG.md#v230-2024-10-23

Note: This is a repeat of jazzband#257, since that PR was lost as part
of merges that occurred during the release of 3.0.0:
jazzband#257 (comment)
@codecov
Copy link

codecov bot commented Jul 1, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (7c47649) to head (3c70fe4).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #269   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines          131       130    -1     
  Branches        14        14           
=========================================
- Hits           131       130    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@edmorley
Copy link
Contributor Author

edmorley commented Jul 1, 2025

@palfrey @mattseymour Please could I have review/merge for this PR, which is a re-apply of #257, which appears to have been inadvertently overwritten during the merges for the recent 3.0.0 release? Thanks :-)

@palfrey palfrey merged commit ae2fabf into jazzband:master Jul 1, 2025
12 checks passed
@edmorley
Copy link
Contributor Author

edmorley commented Jul 2, 2025

@palfrey Thank you for the review/merge :-) I don't suppose you could release a v3.0.1 with this change? (Given the v3.0.0 GitHub releases item description says it's already shipped)

@mattseymour
Copy link
Contributor

Released

@edmorley
Copy link
Contributor Author

edmorley commented Jul 2, 2025

@mattseymour Many thanks!

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