Skip to content
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

Add Python 3.12 Support #752

Merged
merged 7 commits into from
Nov 15, 2023
Merged

Conversation

anton-petrov
Copy link

@anton-petrov anton-petrov commented Oct 5, 2023

Description

Added support for Python 3.12

All tests passed ✌️

Ready for merge.

Pinned dependencies:

  • cython==0.29.36
  • pydantic==1.10.13
  • flask==2.1.3
  • werkzeug==2.2.2
  • aiohttp==3.9.0b1

Fixed Github Actions for CI and tox configuration.
Removed tests for legacy python versions, supported only CPython >= 3.8 and PyPy >= 3.9

@batazor
Copy link

batazor commented Oct 9, 2023

@boonya @ustims @rmk135 Please take a look when you have the opportunity

@abdullah-retorio
Copy link

Hello,

I am planning to use your package in my new project. But i saw that you fixed the pydantic version in requirement-dev.txt. I need to use pydantic V2. Does this fix block my usage? If yes, will you support pydantic v2 and when?

@anton-petrov
Copy link
Author

@abdullah-retorio Hi, pydantic version doesn't matter, it's only for tests. I successfully use pydanticV2 and this DI in my project.

@macieyng
Copy link

macieyng commented Nov 9, 2023

Hi 👋
When can we expect this to be merged?
aio-libs/aiohttp#7639 seems to be resolved.
Linking issue #737.

@anton-petrov
Copy link
Author

@rmk135 Hi, please, could you review?

@zeshuaro
Copy link

You might also want to update the CI config to also run tests on Python 3.12: https://github.com/ets-labs/python-dependency-injector/blob/master/.github/workflows/tests-and-linters.yml

@anton-petrov
Copy link
Author

The reason why I deleted tests for an older python versions is - СPython version 3.7 and lower is no longer supported.

@rmk135 rmk135 changed the base branch from master to feature/python-3.12 November 15, 2023 01:24
@rmk135 rmk135 self-assigned this Nov 15, 2023
@rmk135 rmk135 added the feature label Nov 15, 2023
@rmk135
Copy link
Member

rmk135 commented Nov 15, 2023

@anton-petrov thanks a lot for the PR. For some reason, some of the tests are not passing. Also, I probably would keep Python 3.7 support for a little bit because it's still around 5% of users who haven't upgraded just yet.

I'm going to merge the PR as is into a feature branch and continue working on it there.

@rmk135 rmk135 merged commit 437300e into ets-labs:feature/python-3.12 Nov 15, 2023
7 of 16 checks passed
@mhassan5809
Copy link

hello @rmk135 when this pr will get merge to main branch .
Thanks .

@nerlijman
Copy link

Is there a new tag containing this ?

@brunns brunns mentioned this pull request Jun 5, 2024
@fgreinacher
Copy link

@rmk135 Sorry for the ping, but it would be awesome if you could create a new release with this change. Also happy to help if some preparation work is needed. Just let me know 🙇

@macieyng
Copy link

@fgreinacher see #765

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants