-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added ruff for pre-commit check #885
Conversation
- added repo to .pre-commit-config.yaml - added rules to pyproject.toml - removed black, because ruff has drop-in parity with it
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the defaults do you find that ads builds fine? I found that I had to use the ruff conf that I shared to exclude various things otherwise it would flag a number or issues. If you don't find that then go ahead. Thanks for finishing this.
- added ignore and extended-select - added types_or: [ python, pyi, jupyter ] to config
|
|
Description
Ruff can be used to replace Flake8 (plus dozens of plugins), Black, isort, pydocstyle, and more, all while executing tens or hundreds of times faster than any individual tool.
Added ruff to run mainly default settings with little extra and exclude. While developing our code we will add/exclude more. Please check on rules and suggest what can be added to default (section 'select' in pyproject.toml) and what can be excluded (section 'ignore' in pyproject.toml)- https://docs.astral.sh/ruff/rules/.
What was done