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

Remove code for deprecation of Context keys #45585

Merged
merged 1 commit into from
Jan 12, 2025

Conversation

kaxil
Copy link
Member

@kaxil kaxil commented Jan 12, 2025

We removed all the deprecated keys in #43902 so we no longer need this code.

In preparation of #45583, I want to simplify this code. We can always revert/re-add this later when we need to deprecate a key.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@kaxil kaxil force-pushed the remove-context-deprecation branch 3 times, most recently from e70d685 to 8099404 Compare January 12, 2025 12:15
We removed all the deprecated keys in apache#43902 so we no longer need this code.

In preparation of apache#45583, I want to simplify this code. We can always revert/re-add this later when we need to deprecate a key.
@kaxil kaxil force-pushed the remove-context-deprecation branch from 8099404 to a8cbcd1 Compare January 12, 2025 12:38
@kaxil kaxil merged commit 5597bb2 into apache:main Jan 12, 2025
62 checks passed
@kaxil kaxil deleted the remove-context-deprecation branch January 12, 2025 13:15
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.

3 participants