-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
chore: ruff cleanup #2779
chore: ruff cleanup #2779
Conversation
Why not, there is some examples here: https://github.com/astral-sh/ruff?tab=readme-ov-file#installation We just have to take into account the following thinks in this PR:
|
exclude already ignores rules with flake8
@nicolargo I've just enabled some of the easy to fix lint rules that should not change any behavior. I think it would be better to enable the others slowly over other releases as they would require more invasive code changes. |
… the venv* folders in the linter configuration
Review ok on my side, i have to reconfigure my VSCode to set Ruff as the default Linter (instead of Flake8). I just push some minors changes. PR can be pushed on develop. |
Fixes all formatting by ruff
@nicolargo
Should we add something like precommit tooling that automatically runs ruff on each and every commit?
Something like would be nice: https://github.com/pre-commit/pre-commit