Skip to content

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) / edge3 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
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Apr 4, 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
@jscheffl jscheffl deleted the feature/clean-leftovers-of-removed-in-airflow-3-warning branch October 5, 2025 07:40
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) / edge3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants