-
Notifications
You must be signed in to change notification settings - Fork 101
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
Replace isort, black and flake8 with ruff #617
Replace isort, black and flake8 with ruff #617
Conversation
Ruff is great, especially for linting. I have a slight concern about switching over from black/isort given the [minor] differences, meaning that IDEs then need to be reconfigured to not use black/isort and use ruff instead, and I'm not sure ruff is currently used more widely than black for formatting. Otherwise I'm in favour, with the biggest win being |
Yeah I can revert to still using black for formatting. It does not seem like there is any changes related to ruff isort compared to "normal" isort, so tempted to keep that to reduce the number of dependencies if that is okay :) |
I'm in favor of using ruff for everything (isort, flake8, and black replacement). While it is true that it is not as used as black and isort, there is a big win in developer experience. For those who don't wish to use the pre-commit for example. It's true that people will have to change the formatting in their IDE for those who use it, but if the pre-commit is installed anyway, the formatting will be fixed before commiting. It was long overdue that the Python ecosystem get a rust-like developer experience with tooling. A question I have. Will |
I think so. Will update :) |
Thanks! Great PR by the way :) |
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.
Looks good, thanks!
9874b40
into
gabrieldemarmiesse:master
Ruff has emerged as the most popular tool for linting, and covers the rules from flake8 among others. It also serves as an (almost) drop in replacement for black and isort.
This PR replaces flake8, isort and black with ruff.
F
,E
andW
covers the rules by the existing flake8 according to these docs