Skip to content

Rename airflowctl assets get-by-alias to get-alias#70453

Draft
justinpakzad wants to merge 1 commit into
apache:mainfrom
justinpakzad:fix/get-alias-ctl
Draft

Rename airflowctl assets get-by-alias to get-alias#70453
justinpakzad wants to merge 1 commit into
apache:mainfrom
justinpakzad:fix/get-alias-ctl

Conversation

@justinpakzad

@justinpakzad justinpakzad commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

The existing command to fetch an asset alias by it's ID was called get-by-alias, implying we are fetching an asset by it's alias. This is a bit misleading as it's actually fetching an asset alias by it's ID. This PR updates the name of the command to get-alias to better reflect what the command is actually doing. I have also updated the existing unit test and help text to align with the new naming/description.

Was generative AI tooling used to co-author this PR?
  • No (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.

@justinpakzad justinpakzad changed the title Fix airflowctl get asset alias command Rename airflowctl assets get-by-alias to get-alias Jul 26, 2026
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.

1 participant