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.
--
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]
