-
Notifications
You must be signed in to change notification settings - Fork 79
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
add ruff
and remove redundant pre-commits
#278
Conversation
I think |
now there are |
Perfect, can you create another issue after this one is merged for this? |
* fix D101 and B028 (stack level for warnings) introduced in #219 * add vscode to gitignore * fix D102 on costs.py and try to simplify inheritance methods * try to fix D102 everywhere * fix D103 * remove comment from .flake8 * add ruff to pyproject.toml * add ruff * add documentation on ruff select and add unfixable * add target version * add comment on tool.ruff * pyproject.toml * format * remove artifcats from wrong merge * address ruff comments * fix ruff imports
replace
flake8
and a bunch of other pre-commit with https://github.com/charliermarsh/ruff, a faster alternative.If accepted/in scopre, merge only after #269