-
Notifications
You must be signed in to change notification settings - Fork 48
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
Pydantic 2.4 support #59
Pydantic 2.4 support #59
Conversation
…ncies section, disable apache-airflow (needs to be manually side loaded as poetry is unable to)
updates:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #59 +/- ##
=======================================
Coverage 88.49% 88.49%
=======================================
Files 22 22
Lines 3189 3189
=======================================
Hits 2822 2822
Misses 367 367 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Please let me know when/if we can get a release around this PR. :) |
One more PR pending from @stikkireddy , as it gets fixed we can do 1.0.0. We can target next week |
Description
Disable apache-airflow (needs to be manually side loaded as poetry is unable to) - this because apache-airflow supports pydantic 2.*, but poetry's dependency resolced doesn't understand how to resolve it correctlyBlack configuration modernized (120 character width vs 88)Related Issue
N/A
Motivation and Context
Want to bring in support for pydantic 2.4
How Has This Been Tested?
Unit tests and some integration tests with locally available projects that run against Brickflow
Screenshots (if appropriate):
N/A
Types of changes
Checklist:
My change requires a change to the documentation.