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

Move Literal alias into TYPE_CHECKING block #45345

Merged
merged 2 commits into from
Jan 9, 2025

Conversation

uranusjr
Copy link
Member

@uranusjr uranusjr commented Jan 2, 2025

This allows us to always import Literal from typing, which has received quite some bug fixes between different versions. Doing this in the typing block avoids loading the runtime version at all, thus eliminating differences between runtime Python versions.

@potiuk
Copy link
Member

potiuk commented Jan 2, 2025

@uranusjr - can you please rebase that one -> we found and issue with @jscheffl with the new caching scheme - fixed in #45347 that would run "main" version of the tests. I am asking in all affected PRs to rebase.

@uranusjr uranusjr force-pushed the improve-literal branch 3 times, most recently from 9f9b343 to 8d236ad Compare January 2, 2025 14:44
This allows us to always import Literal from typing, which has received
quite some bug fixes between different versions. Doing this in the
typing block avoids loading the runtime version at all, thus eliminating
differences between runtime Python versions.
@uranusjr uranusjr merged commit b703d53 into apache:main Jan 9, 2025
91 checks passed
@uranusjr uranusjr deleted the improve-literal branch January 9, 2025 05:34
agupta01 pushed a commit to agupta01/airflow that referenced this pull request Jan 13, 2025
karenbraganz pushed a commit to karenbraganz/airflow that referenced this pull request Jan 13, 2025
HariGS-DB pushed a commit to HariGS-DB/airflow that referenced this pull request Jan 16, 2025
got686-yandex pushed a commit to got686-yandex/airflow that referenced this pull request Jan 30, 2025
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.

5 participants