MonkeyCanCode commented on issue #2419: URL: https://github.com/apache/polaris/issues/2419#issuecomment-3213043919
Thanks for bring this up @flyrain. Here are my two cents: Before we can do so, we should fix couple old issues we have with client code such as following 1. remove doc generation from openapi as those will get include as those are not really useful 2. hide the lengthy "[main] ERROR o.o.codegen.DefaultCodegen - Required var xxx not in properties" when invoking openapi to generate client code 3. decide what to publish (sdist or wheel or both?). see my sample PR in https://github.com/apache/polaris/pull/2425 4. remove the needed of `SCRIPT_DIR` when invoking `polaris` as we should really using <home_dir>/.polaris instead imo (see previous discussion in https://github.com/apache/polaris/pull/931) 5. rename bash script `polaris` on the root directory to something else such as client_setup.sh etc. to avoid confusion as now the client python will create a `polaris` binary file. cc. @eric-maynard @HonahX -- 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
