MonkeyCanCode commented on PR #1995:
URL: https://github.com/apache/polaris/pull/1995#issuecomment-3047269631
> > @MonkeyCanCode Thanks for improving the Makefile! Overall LGTM
> > There are some discussion around doing the code check/format in a
different way: #1986. Since the current one is an improvement of the existing
solution, I suggest we seek to merge this first and change later based on the
result of #1986
>
> Exactly, it is a lot harder to do without this piece first (thinking about
using native shell and markfile route instead of a third party plugin which may
not be supported after 1-2 years).
@HonahX please take another look when you get a chance. i added a helper for
Makefile as well (update the PR summary to reflect the same):
```
➜ python git:(fix_python_makefile) make
Usage:
make <target>
help Display this help.
version Print version information.
regenerate-client Regenerate the client code
test-integration Run integration tests
lint Run linting checks
clean Cleanup
```
--
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
