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

Added support for pydantic-settings in Configuration.from_pydantic() #768

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

tsaryapkin
Copy link

  • Configuration provider can be initialised via from_pydantic method from both pydantic.BaseSettings (pydantic v1) and pydantic_settings.BaseSettings (based on pydantic v2).
  • I also specified flask and Werkzeug (used by flask) versions because the latest version of flask seems to be incompatible with the library and lots of tests failed.

@apollo17m
Copy link

Hey @tsaryapkin , @rmk135

@tsaryapkin Thanks a lot for this update, I'm currently blocked from using this lib with a new project because of this issue and this would help a lot.

@rmk135 Is it possible to review this PR soon ?

Thanks !

@martinkozle
Copy link

Hey @tsaryapkin , @rmk135

@tsaryapkin Thanks a lot for this update, I'm currently blocked from using this lib with a new project because of this issue and this would help a lot.

@rmk135 Is it possible to review this PR soon ?

Thanks !

I second this!

@zahycs
Copy link

zahycs commented Dec 18, 2023

Hi @rmk135

Any plans to merge this soon?

Thank you

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.

None yet

5 participants