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.

Sign up for GitHub

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

Jump to bottom

Cleanup Context type hinting for Task SDK #45454

Closed
kaxil opened this issue Jan 7, 2025 · 0 comments · Fixed by #45657
Closed

Cleanup Context type hinting for Task SDK #45454

kaxil opened this issue Jan 7, 2025 · 0 comments · Fixed by #45657
Assignees
@kaxil
Labels
area:dev-tools

Comments

@kaxil
Copy link
Member

kaxil commented Jan 7, 2025

I have change typehints from Context to Mapping[str, Any] in #45444. Let's make it consistent -- the Context class currently manages deprecations too which we haven't ported to Task SDK yet.
@kaxil kaxil self-assigned this Jan 7, 2025
@kaxil kaxil moved this from Next 2 Weeks to In Progress in AIP-72 - Task Execution Interface and SDK Jan 14, 2025
kaxil added a commit to astronomer/airflow that referenced this issue Jan 14, 2025
@kaxil
Add other instances of Context type hints
e655f25 
Follow-up of apache#45583

closes apache#45454
@kaxil kaxil mentioned this issue Jan 14, 2025
Merged
kaxil added a commit to astronomer/airflow that referenced this issue Jan 15, 2025
@kaxil
Add other instances of Context type hints
a7ce6b7 
Follow-up of apache#45583

closes apache#45454
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kaxil kaxil

Labels
area:dev-tools
Projects
AIP-72 - Task Execution Interface and SDK
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add other instances of Context type hints astronomer/airflow
1 participant
@kaxil