-
-
Notifications
You must be signed in to change notification settings - Fork 18k
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
Bump ruff to latest version #57766
Bump ruff to latest version #57766
Conversation
@mroeschke I wonder if it's reasonable to deprecate EX03 in favor of If I understand correctly, EX03 is used to validate that a code block in a docstring follows PEP8. Currently EX03 is using In my opinion, we can just use |
I think there's momentum towards removing EX03. We can't right now since some docstrings are dynamically generated and |
@mroeschke how about adding E704 (Multiple statements on one line (def)) to the list of ignored errors for EX03? Currently there are |
If that's the code that conflict with ruff's formating, that sounds good |
@mroeschke this PR should be ready for review, apart from the second commit, which is automatic, the rest are manual small fixes |
Thanks @tqa236 |
* Bump ruff to latest version * Autoformat * Ignore conflicted error * Ignore false positive
Bump
ruff
to the latest version in a separate PR because of the big formatting change.