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

Reply via email to