Opened at 2023-04-17T14:00:25Z

Closed at 2023-04-25T12:37:06Z

#4014 closed enhancement (fixed)

Remove pylint from codechecks

Reported by: itamarst Owned by: GitHub <noreply@…>
Priority: normal Milestone: undecided
Component: unknown Version: n/a
Keywords: Cc:
Launchpad Bug:

Description

On my computer (which is twice as fast as most hosted CI runners!) running pylint takes 18 seconds, for the one check we use it for.

In contrast, the same check with ruff takes 60ms.

Running _every single ruff check_, which is flake8 and some of pylint and many more checks takes 290ms (or 1.5 seconds on single core).

A minimal first step then is replacing pylint with ruff, but probably we should swap out flake8 as well.

Change History (1)

comment:1 Changed at 2023-04-25T12:37:06Z by GitHub <noreply@…>

  • Owner set to GitHub <noreply@…>
  • Resolution set to fixed
  • Status changed from new to closed

In 4cdd448/trunk:

Merge pull request #1290 from tahoe-lafs/4014-remove-pylint

Remove pylint and flake8, replacing with ruff

Fixes ticket:4014

Note: See TracTickets for help on using tickets.