Eason09053360 opened a new pull request, #72607:
URL: https://github.com/apache/airflow/pull/72607
The `--columns` help for `airflow dags list` said the default was
`['dag_id', 'fileloc', 'owner', 'is_paused']`, but the actual default has been
`('dag_id', 'fileloc', 'owners', 'is_paused', 'bundle_name', 'bundle_version')`
since #45779, and the valid field name has always been `owners` (the help said
`owner` since the option was introduced in #35250).
Following the help literally (`--columns owner`) prints an "Ignoring the
following invalid columns" error and drops the column from the table. The
generated CLI reference docs (`cli-and-env-variables-ref.rst`, built via `..
argparse::`) carried the same wrong text.
This PR derives the help string from the same tuple used as `default`, so
the two cannot drift apart again, and adds a parametrized test that asserts the
`(default: [...])` fragment in the help of both `--columns` arguments (`dags
list`, `assets list`) matches their actual `default`.
Rendered help after the change:
```
--columns COLUMNS List of columns to render. (default: ['dag_id',
'fileloc', 'owners', 'is_paused', 'bundle_name', 'bundle_version'])
```
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Claude Code (Fable 5.1)
Generated-by: Claude Code (Fable 5.1) following [the
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
