eric-maynard commented on code in PR #2154: URL: https://github.com/apache/polaris/pull/2154#discussion_r2223363658
########## client/python/cli/options/parser.py: ########## @@ -57,22 +57,24 @@ class Parser(object): ), Argument(Arguments.PROFILE, str, hint="profile for token-based authentication"), Argument(Arguments.PROXY, str, hint="proxy URL"), + Argument(Arguments.DEBUG, bool, hint="Enable debug mode"), ] @staticmethod def _build_parser() -> argparse.ArgumentParser: parser = TreeHelpParser(description="Polaris CLI") for arg in Parser._ROOT_ARGUMENTS: + kwargs = {"help": arg.hint} if arg.default is not None: - parser.add_argument( Review Comment: But it looks like we only changed it for the root arguments, while the other boolean arguments (e.g. `CASCADE`) are unchanged -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org