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

Clean Leftovers of RemovedInAirflow3Warning #47264

Conversation

jscheffl
Copy link
Contributor

@jscheffl jscheffl commented Mar 1, 2025

There was only one deprecation left - this PR cleans the RemovedInAirflow3Warning from the codebase...

Found still some dependencies and attempted to clean behind. Would request 4 eyes to review.

@jscheffl jscheffl added the full tests needed We need to run full set of tests for this PR to merge label Mar 1, 2025
@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:dev-tools area:Executors-core LocalExecutor & SequentialExecutor area:Scheduler including HA (high availability) scheduler area:serialization kind:documentation provider:edge Edge Executor / Worker (AIP-69) labels Mar 1, 2025
@jscheffl jscheffl force-pushed the feature/clean-leftovers-of-removed-in-airflow-3-warning branch from 18cad63 to 03a795a Compare March 1, 2025 22:52
@jscheffl jscheffl force-pushed the feature/clean-leftovers-of-removed-in-airflow-3-warning branch from 03a795a to 47756b6 Compare March 3, 2025 13:58
@jscheffl jscheffl force-pushed the feature/clean-leftovers-of-removed-in-airflow-3-warning branch 2 times, most recently from 189f687 to 3796c13 Compare March 4, 2025 22:36
@jscheffl jscheffl marked this pull request as ready for review March 5, 2025 08:40
@eladkal eladkal added the airflow3.0:breaking Candidates for Airflow 3.0 that contain breaking changes label Mar 5, 2025
Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few nits / suggestions

@jscheffl jscheffl force-pushed the feature/clean-leftovers-of-removed-in-airflow-3-warning branch from 3796c13 to f27900d Compare March 5, 2025 10:43
Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good thanks :)

@jscheffl jscheffl merged commit bb8d465 into apache:main Mar 5, 2025
89 checks passed
shahar1 pushed a commit to shahar1/airflow that referenced this pull request Mar 5, 2025
* Clean Leftovers of RemovedInAirflow3Warning

* Add newsfragment for deprecations

* Revert one cleanup in decorator

* Replace further leftovers of airflow.api.auth.backend.session in code

* Review feedback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
airflow3.0:breaking Candidates for Airflow 3.0 that contain breaking changes area:API Airflow's REST/HTTP API area:dev-tools area:Executors-core LocalExecutor & SequentialExecutor area:Scheduler including HA (high availability) scheduler area:serialization full tests needed We need to run full set of tests for this PR to merge kind:documentation provider:edge Edge Executor / Worker (AIP-69)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants