integreat-cms icon indicating copy to clipboard operation
integreat-cms copied to clipboard

Replace pylint + isort by ruff

Open timobrembeck opened this issue 1 year ago • 2 comments

Motivation

Our CI workflows take quite a while, mostly due to the tests, but also pylint is taking a considerable amount of time. I think we could reduce this by migrating from pylint to ruff:

232603514-c95e9b0f-6b31-43de-9a80-9e844173fd6a

Proposed Solution

  • Replace pylint and isort by ruff
  • Check whether there's an automated tool which can migrate the pylint: disable= statements

Alternatives

Maybe it isn't worth the effort...

Additional Context

Pylint is not completely implemented by ruff yet (see https://github.com/astral-sh/ruff/issues/970), so we would loose a few rules (but also ruff has a lot of extra checks that pylint doesn't, mostly those of flake8)...

timobrembeck avatar Oct 09 '23 11:10 timobrembeck