-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
fix: add pytz dependency #7077
fix: add pytz dependency #7077
Conversation
e1390d7
to
81d3b13
Compare
Thanks for the quick work @sdebruyn! We'll need to backport this for inclusion in patch releases of actively supported versions. |
@jtcohen6 already on it :) |
Amazing - thank you! I think my US-based colleagues are still asleep. As soon as someone is online, let's get these reviewed & merged & patched & released :) |
if you'd be wondering where I got the version constraint from, this is what Babel has so what dbt-core previously had implicitly |
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.
Look goods. Thanks for the quick work here!
Thanks for jumping on this so quickly, we spotted this issue this morning impacting some of our ci pipelines |
resolves #7075
Description
dbt-core uses pytz directly. The dependency on babel guaranteed pytz to be there until now, but that dependency was recently removed. Since dbt-core uses pytz directly (in tracking), it should explicitly add a dependency on it
Checklist
changie new
to create a changelog entry