-
Notifications
You must be signed in to change notification settings - Fork 1
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
Introduce ruff #363
Introduce ruff #363
Conversation
on certain lines due to conflict with pyright
is it possible to change the displayed name from Test to Ruff? |
The only way I can see is to put Ruff into a separate job |
I agree with @disrupted that we should gradually introduce the changes (fixes) made by Ruff. Maybe it would be good to ignore most of the (critical) rules at first and do them step by step in follow-up PRs to avoid breaking the code. |
I will look through all of the changes today and add comments if I run into particularly big changes. otherwise, of course it's best practice to bundle formatting changes in one big commit since it's easier to exclude from git blame. |
ok, then it's fine |
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 great 🤩
(almost) no complaints :)
Replaces
isort
,pyupgrade
,flake8
withruff