Acehaidrey commented on a change in pull request #18916:
URL: https://github.com/apache/airflow/pull/18916#discussion_r734124385
##########
File path: airflow/cli/cli_parser.py
##########
@@ -694,6 +694,10 @@ def _check(value):
help="Don't start the serve logs process along with the workers",
action="store_true",
)
+ARG_ROLE_IMPORT = Arg(("file",), help="Import roles from JSON file",
nargs=None)
+ARG_ROLE_EXPORT = Arg(("file",), help="Export all roles to JSON file",
nargs=None)
+ARG_ROLE_EXPORT_FMT = Arg(('-p', '--pformat'), action='store_true',
+ help='Format output JSON file by sorting role names
and indenting')
Review comment:
is it okay with you to just name it pretty and apply both? I think
having two separate options here may be little much since I don't imagine it
will be thought about too much.
--
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]
