Skip to content

Removed support for Python 3.8#1266

Merged
sliverc merged 3 commits into
django-json-api:mainfrom
n2ygk:deprecate_3.8
Jan 13, 2025
Merged

Removed support for Python 3.8#1266
sliverc merged 3 commits into
django-json-api:mainfrom
n2ygk:deprecate_3.8

Conversation

@n2ygk

@n2ygk n2ygk commented Jan 7, 2025

Copy link
Copy Markdown
Contributor

Description of the Change

Remove Python 3.8 which is EOL per PEP 569

Checklist

  • PR only contains one change (considered splitting up PR)
  • unit-test added
  • documentation updated
  • CHANGELOG.md updated (only for user relevant changes)
  • author name in AUTHORS

@n2ygk n2ygk requested a review from sliverc January 7, 2025 16:38
@n2ygk

n2ygk commented Jan 7, 2025

Copy link
Copy Markdown
Contributor Author

drfmaster tests are throwing an error for TypeError for encode of None. Maybe related to #1229?

@sliverc

sliverc commented Jan 7, 2025

Copy link
Copy Markdown
Member

There is an upstream change in DRF which caused these errors. I have submitted a PR here encode/django-rest-framework#9615 hope it will be merged soon.

@sliverc sliverc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for working on this. getting-started.md needs also to be adjusted I think and also add a note in changelog to version 7.1.0 that states that it is the last version supporting 3.7. See how we did it in #1178

@n2ygk

n2ygk commented Jan 9, 2025

Copy link
Copy Markdown
Contributor Author

@sliverc

Thanks for working on this. getting-started.md needs also to be adjusted I think and also add a note in changelog to version 7.1.0 that states that it is the last version supporting 3.7. See how we did it in #1178

Done.

Also, perhaps as a separate PR, should we make a failed drfmaster test a non-fatal workflow error? That's how we do it in DOT, for example: https://github.com/jazzband/django-oauth-toolkit/blob/master/tox.ini#L57-L59. Right now, errors in non-released DRF are blocking successful PRs here.

@n2ygk

n2ygk commented Jan 9, 2025

Copy link
Copy Markdown
Contributor Author

See #1267

@sliverc

sliverc commented Jan 13, 2025

Copy link
Copy Markdown
Member

Thanks looks good.

@sliverc sliverc changed the title Python 3.8 EoL per PEP569 Removed support for Python 3.8 Jan 13, 2025
@sliverc sliverc merged commit 1b5fb9c into django-json-api:main Jan 13, 2025
@n2ygk n2ygk deleted the deprecate_3.8 branch January 13, 2025 14:34
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.

2 participants