Skip to content

Unify ZoneInfo.from_file signature #135273

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

Open
hrimov opened this issue Jun 8, 2025 · 0 comments
Open

Unify ZoneInfo.from_file signature #135273

hrimov opened this issue Jun 8, 2025 · 0 comments
Labels
stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error

Comments

@hrimov
Copy link
Contributor

hrimov commented Jun 8, 2025

Bug report

Bug description:

While enhancing stubs for ZoneInfo.from_file in typeshed, I noticed that the regression in the from_file signature was caused by the argument clinic migration, but without specifying an argument alias - intentionally or not, I don't know.

Therefore, I believe it would be a good approach to unify both signatures for the native and Python versions of zoneinfo. Since AC takes priority and fobj is a pos-only argument, I think the more appropriate approach would be to update Python's fallback.

CPython versions tested on:

CPython main branch

Operating systems tested on:

macOS

Linked PRs

@hrimov hrimov added the type-bug An unexpected behavior, bug, or error label Jun 8, 2025
@picnixz picnixz added the stdlib Python modules in the Lib dir label Jun 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stdlib Python modules in the Lib dir type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

No branches or pull requests

2 participants