MonkeyCanCode commented on PR #1954: URL: https://github.com/apache/polaris/pull/1954#issuecomment-3029482370
> > > @MonkeyCanCode what exactly did you run here btw? I still see conflicts/changes on #1912 after running `ruff check client/python/cli --fix` there. > > > > > > For the above, I am using `uv` to setup `ruff` as `ruff` is not part of the current tool we used. You can do the same with following instructions: > > ``` > > ➜ polaris git:(main) cd client > > ➜ client git:(main) uv add --dev ruff > > ➜ client git:(main) uv run ruff format > > ``` > > > > > > > > > > > > > > > > > > > > > > > > I will look into on how we can automate this tomorrow. > > @eric-maynard sample PR is ready: #1986 > > let me know what you think. Thanks. @eric-maynard after https://github.com/apache/polaris/pull/1995 is merged, u can just run `make lint` for this to work without tedious instructions above. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
