codeworm96 commented on issue #3468: URL: https://github.com/apache/incubator-opendal/issues/3468#issuecomment-1811650909
Sure. I would like to share my current plan for discussion before submitting the PRs: 1. we need to add ruff (and ruff confs) to the project: It seems currently dev dependencies as organized as optional-dependencies, like `benchmark`, `docs`, `tests`. I think we can add a new `lint` category for ruff. 2. I tested the current code, it did not pass the lint by default. So some code change and/or suppressing some rules are needed. 3. As the final step, I think we should add lint as part of the CI tests once we fix the lint. But I do not have a detailed idea yet. -- 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]
