Skip to content

For now limit Pydantic to < 2.0.0#33235

Merged
potiuk merged 2 commits into
apache:mainfrom
potiuk:limit-pydantic-to-2
Aug 8, 2023
Merged

For now limit Pydantic to < 2.0.0#33235
potiuk merged 2 commits into
apache:mainfrom
potiuk:limit-pydantic-to-2

Conversation

@potiuk

@potiuk potiuk commented Aug 8, 2023

Copy link
Copy Markdown
Member

Pydantic 2 adds few warnings/deprecations and until AWS dependencies are updated, we cannot move to it anyway.

Limiting Pydantic to <2 for now should prevent the warnings and heep our dependencies in check.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

Pydantic 2 adds few warnings/deprecations and until AWS dependencies
are updated, we cannot move to it anyway.

Limiting Pydantic to <2 for now should prevent the warnings and
heep our dependencies in check.
Comment thread setup.cfg Outdated
Co-authored-by: Jed Cunningham <[email protected]>
@potiuk
potiuk merged commit a752c90 into apache:main Aug 8, 2023
@potiuk
potiuk deleted the limit-pydantic-to-2 branch August 8, 2023 22:29
@ephraimbuddy ephraimbuddy added this to the Airflow 2.7.0 milestone Aug 9, 2023
@ephraimbuddy ephraimbuddy added the type:misc/internal Changelog: Misc changes that should appear in change log label Aug 9, 2023
ephraimbuddy pushed a commit that referenced this pull request Aug 9, 2023
* For now limit Pydantic to < 2.0.0

Pydantic 2 adds few warnings/deprecations and until AWS dependencies
are updated, we cannot move to it anyway.

Limiting Pydantic to <2 for now should prevent the warnings and
heep our dependencies in check.

* Update setup.cfg

Co-authored-by: Jed Cunningham <[email protected]>

---------

Co-authored-by: Jed Cunningham <[email protected]>
(cherry picked from commit a752c90)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:misc/internal Changelog: Misc changes that should appear in change log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants