MonkeyCanCode opened a new pull request, #2192:
URL: https://github.com/apache/polaris/pull/2192

   This is a follow up PR based on https://github.com/apache/polaris/pull/2049. 
To give more context, currently we are using `client/templates/regenerate.sh` 
to regenerate code for python cli from openapi within docker. Then on gradle 
side, we have a CMD registers as `regeneratePythonClient` to provide user to 
regenerate the python code.
   
   Based on the discussion from above PR as well as the experimental PR 
(https://github.com/apache/polaris/pull/1885) from @HonahX, I then raised this 
PR to move python code generation away from gradle and make it as part of 
poetry and remove dependency for docker. 
   
   Unlikes the experimental PR, this PR has the following as extra:
   1. suppress noise stdout when invoking openapi code generation
   2. ensure openapi only generate source code and another else (the one in the 
experimental PR will setup a bunch files and try to modify many existed files 
as well including pyproject.toml and README.md)
   3. makefile and polaris bash script are updated to not depends on gradle for 
client code generation
   4. remove client/templates/regenerate.sh as that is no longer needed (all 
functionalities are ported to the build.py)
   5. 


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