Skip to content

gh-114713: Revert gh-114731 #133330

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

Merged
merged 1 commit into from
May 3, 2025
Merged

gh-114713: Revert gh-114731 #133330

merged 1 commit into from
May 3, 2025

Conversation

Eclips4
Copy link
Member

@Eclips4 Eclips4 commented May 3, 2025

Unfortunately, #114731 introduced a regression and zoneinfo.ZoneInfo(None) now raises a ValueError instead of TypeError, which may break some tests in 3rd party libraries (such as https://github.com/reagento/adaptix/).

Therefore, I'm reverting it before 3.14b1. The original issue isn't critical, and we can address it later.

@Eclips4 Eclips4 changed the title gh-114731: Revert gh-114713 gh-114713: Revert gh-114731 May 3, 2025
@Eclips4 Eclips4 added the needs backport to 3.13 bugs and security fixes label May 3, 2025
@Eclips4 Eclips4 enabled auto-merge (squash) May 3, 2025 07:26
@Eclips4 Eclips4 merged commit fe44fc4 into python:main May 3, 2025
45 checks passed
@miss-islington-app
Copy link

Thanks @Eclips4 for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 3, 2025
Revert "pythongh-114713: Handle case of an empty string passed to `zoneinfo.ZoneInfo` (pythonGH-114731)"

This reverts commit 884df11.
(cherry picked from commit fe44fc4)

Co-authored-by: Kirill Podoprigora <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented May 3, 2025

GH-133331 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label May 3, 2025
Eclips4 added a commit that referenced this pull request May 3, 2025
gh-114713: Revert gh-114731 (GH-133330)

Revert "gh-114713: Handle case of an empty string passed to `zoneinfo.ZoneInfo` (GH-114731)"

This reverts commit 884df11.
(cherry picked from commit fe44fc4)

Co-authored-by: Kirill Podoprigora <[email protected]>
@Eclips4 Eclips4 deleted the revert-zoneinfo branch May 3, 2025 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant