valentinDruzhinin commented on code in PR #46799:
URL: https://github.com/apache/airflow/pull/46799#discussion_r1957329500
##########
airflow/cli/commands/remote_commands/variable_command.py:
##########
@@ -86,6 +86,14 @@ def variables_import(args, session):
var_json = json.load(varfile)
except JSONDecodeError:
raise SystemExit("Invalid variables file.")
+ var_descriptions = {}
+ if args.description_file and os.path.exists(args.description_file):
Review Comment:
Hi @bugraoz93 . I made a separate file for description to not break the
existing file structure.
Right now we have var file as key-value json file. If we add the description
there, I have to change the structure of the file to be smt like `key->{value:
<value>, description: <description>}`. In that case we will break the existing
var file structure. That's why I decided to add a separate file for description.
If we need to update the structure of the var file, let me know, I can
update it.
--
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]
