-
Notifications
You must be signed in to change notification settings - Fork 14.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow 'airflow variables export' to print to stdout #33279
Conversation
476257d
to
619752b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Has the documentation been updated to include this new option for exporting variables? Clear documentation will assist users in utilizing this feature effectively.
Have you checked the help description? Is this is enough or you think about another documentation ? Or maybe you are simply not aware that the help description is automatically reflected in the documentatotion @vijayasarathib ? |
Similar to connections, it is now possible to use '-' to print to stdout. Also includes some refactoring to extract common logic to 'airflow connections export' and improve the code quality.
89ce63b
to
6de6b12
Compare
Co-authored-by: vedantlodha <[email protected]>
This is a continuation to #31854.
Similar to connections, this change allows the CLI to export the variables to stdout using
airflow variables export -
.There are a couple of change aside from the content in the original PR:
is_stdout
function has been extracted and improved.Close #31851.