Skip to content
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

Apply assorted ruff rules #59

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

DimitriPapadopoulos
Copy link
Contributor

No description provided.

F401 imported but unused
F841 Local variable is assigned to but never used
UP010 Unnecessary `__future__` import `print_function` for target Python version
UP025 Remove unicode literals from strings
UP030 Use implicit references for positional format fields
UP031 Use format specifiers instead of percent format
UP032 Use f-string instead of `format` call
PERF401 Use a list comprehension to create a transformed list
B904 Within an `except` clause, raise exceptions with `raise ... from err` or
     `raise ... from None` to distinguish them from errors in exception handling
SIM114 Combine `if` branches using logical `or` operator
SIM118 Use `key in dict` instead of `key in dict.keys()`
C416 Unnecessary `list` comprehension (rewrite using `list()`)
C419 Unnecessary list comprehension
PIE808 Unnecessary `start` argument in `range`
PIE810 Call `endswith` once with a `tuple`
Q000 Single quotes found but double quotes preferred
RSE102 Unnecessary parentheses on raised exception
PLR5501 Use `elif` instead of `else` then `if`, to reduce indentation
NPY201 `np.NZERO` will be removed in NumPy 2.0. Use `-0.0` instead.
RUF019 Unnecessary key check before dictionary access
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant