eric-maynard commented on code in PR #2154:
URL: https://github.com/apache/polaris/pull/2154#discussion_r2242436388


##########
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"),

Review Comment:
   Oh, I see. For some commands it's not easy to support dry-run. I would be 
fine with just failing in that case, or if we want to be really fancy we can 
let the user manually run the command and paste the response into stdin.
   
   Anyway, if this all sounds like too much work let's just skip it -- was only 
an idea.



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