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

chore(deps): Upgrade pytz for python 3.10 compat #27

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ditsuke
Copy link
Contributor

@ditsuke ditsuke commented Jul 27, 2022

What

Upgrades the pytz dependency to the latest version.
This is a Stacked PR stacked atop #26.
Please review in tandem and merge the PRs, without squashing or rebasing, in sequence.

Why

The previously pinned version used an API (collections.Mapping) moved to collections.abc in python3.10.

@engagepy
Copy link
Contributor

I ran it locally, it already is.

@ditsuke
Copy link
Contributor Author

ditsuke commented Jul 27, 2022

ah, it was a python 3.10 change. let me correct

@ditsuke ditsuke changed the title chore(deps): Upgrade pytz for python 3 compat chore(deps): Upgrade pytz for python 3.10 compat Jul 27, 2022
@realslimshanky
Copy link
Owner

Looks like #26 is a predecessor of this, maybe combine changes in this PR with #26?

@ditsuke ditsuke force-pushed the chore/update-dep-pytz branch from 9c53bf9 to 8c5468c Compare August 7, 2022 05:37
@ditsuke
Copy link
Contributor Author

ditsuke commented Aug 7, 2022

rebased, @realslimshanky can we please keep them stacked? To make reviewing easier I think my latest PR can be tested.

Updates pinned version of "requests" for compat with the required
urllib3 version.
@ditsuke ditsuke force-pushed the chore/update-dep-pytz branch from 8c5468c to 93b600a Compare August 7, 2022 05:54
Upgrades pytz to the latest version for python 3 support
@ditsuke ditsuke mentioned this pull request Aug 7, 2022
@realslimshanky
Copy link
Owner

@ditsuke in case we're planning to use stacked PR, you'll have to change merge branch to ditsuke:chore/update-dep-pytz. But that won't work since the branch is not in this repo but it's in the fork. So we can mark this PR blocked. Let #26 merge and then rebase.

@ditsuke ditsuke marked this pull request as draft August 13, 2022 06:25
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.

3 participants