Skip to content

Lint codebase with ruff #199

Open
@Aleshkev

Description

@Aleshkev

As suggested by @twalen, we should try to lint codebase using ruff.

Running it right now yields:

  • 1019 errors with default settings
  • 25791 errors when all checks are enabled (probably not all of them should be enabled, like "S101 Use of assert detected", or enabled as of now, like "PTH123 open() should be replaced by Path.open())
    • ~25650 errors after running black . and ruff . --fix
    • 2276 errors with ignore = ["S101", "PTH", "N806", "ANN", "D"] and test*.py excluded

There tend to be a few warnings with hundreds or thousands of occurences, and many warnings with a few each. It should be discussed if it is worth it to go through all of them.

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions