yunchipang commented on PR #49601: URL: https://github.com/apache/airflow/pull/49601#issuecomment-2843776282
@bugraoz93 I wanted to limit this PR to only transitions the `get-value` config command, as this might get too large and hard to review if all 4 commands (get-value, list, lint, update) are implemented. Unit tests are added, please take a look and let me know your thoughts. I have one concern here, in the old airflow CLI, `get` is not displayed as a command (see below). <img width="855" alt="image" src="https://github.com/user-attachments/assets/0f8f5607-6fe2-4b5d-a435-2e0b2d6833f9"; /> However, with the new auto-gen of commands, `get` will be listed as one of the available commands when I use `airflowctl config -h`. I am not sure is this is intended, or we should find a way to undo that. <img width="433" alt="image" src="https://github.com/user-attachments/assets/7d87eb8a-fb19-422e-be4c-683249d7e68a"; /> thanks a lot! -- 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]
