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