Skip to content

Remove unused/dead code in migration utils#64947

Merged
ferruzzi merged 1 commit intoapache:mainfrom
Dev-iL:2604/rm_dead_mig_utils
Apr 9, 2026
Merged

Remove unused/dead code in migration utils#64947
ferruzzi merged 1 commit intoapache:mainfrom
Dev-iL:2604/rm_dead_mig_utils

Conversation

@Dev-iL
Copy link
Copy Markdown
Collaborator

@Dev-iL Dev-iL commented Apr 9, 2026

While working on improving the migration tests, the LLM noted that most functions in the affected file are unused in the codebase - so they're being removed.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

Copy link
Copy Markdown
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

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

Pending CI agrees.

@Dev-iL
Copy link
Copy Markdown
Collaborator Author

Dev-iL commented Apr 9, 2026

Pending CI agrees.

Do we need to run this for "all versions" or add any other tags so we're more confident about the CI's outcome?

@potiuk
Copy link
Copy Markdown
Member

potiuk commented Apr 9, 2026

Pending CI agrees.

Do we need to run this for "all versions" or add any other tags so we're more confident about the CI's outcome?

Good idea.

@potiuk potiuk added the all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs label Apr 9, 2026
@potiuk potiuk force-pushed the 2604/rm_dead_mig_utils branch from 22b8590 to 45e7f24 Compare April 9, 2026 08:41
@potiuk
Copy link
Copy Markdown
Member

potiuk commented Apr 9, 2026

Added label and rebased to re-trigger the CI.

@ferruzzi ferruzzi merged commit 7348e09 into apache:main Apr 9, 2026
128 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

all versions If set, the CI build will be forced to use all versions of Python/K8S/DBs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants