Closed lucabello closed 1 year ago
The flake8 plugins we use are re-implemented in ruff, which is why they have been removed from the dependencies.
isort is enabled (and used when formatting) through the I001 rule.
I001
The "exclude list" has been updated to omit folders that ruff excludes by default.
Everything else is the same, the only difference being two new rules being ignored (which weren't checked before anyway):
N818: Exception name should be named with an Error suffix
RET504: Unnecessary variable assignment before return statement
The flake8 plugins we use are re-implemented in ruff, which is why they have been removed from the dependencies.
isort is enabled (and used when formatting) through the
I001rule.The "exclude list" has been updated to omit folders that ruff excludes by default.
Everything else is the same, the only difference being two new rules being ignored (which weren't checked before anyway):
N818: Exception name should be named with an Error suffix, which I excluded due to the big refactoring required for it;RET504: Unnecessary variable assignment before return statement, which we do everywhere as it increases readability.