-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Add ruff to the CI for fast linting and formatting #7371
Copy link
Copy link
Closed
Labels
kind/healthFor CI/testing/release process/refactoring/technical debt itemsFor CI/testing/release process/refactoring/technical debt itemspriority/before-1.6Finish before the Cirq 1.6 releaseFinish before the Cirq 1.6 releasetriage/acceptedA consensus emerged that this bug report, feature request, or other action should be worked onA consensus emerged that this bug report, feature request, or other action should be worked on
Metadata
Metadata
Assignees
Labels
kind/healthFor CI/testing/release process/refactoring/technical debt itemsFor CI/testing/release process/refactoring/technical debt itemspriority/before-1.6Finish before the Cirq 1.6 releaseFinish before the Cirq 1.6 releasetriage/acceptedA consensus emerged that this bug report, feature request, or other action should be worked onA consensus emerged that this bug report, feature request, or other action should be worked on
Type
Projects
Status
Done
The ruff tool provides a similar feature set to pylint and is orders of magnitude faster. Currently a full check of Cirq sources with
check/pylinttakes about 60 seconds, but is instantaneous withruff.ruffcan also do automated fixes for issues found, which is not possible with pylint.TODO
ruff checkincorporate ruff rules from pyle (we may skip some should they turn out too tedious to adopt)(moved to Enable more ruff rules for the CI #7505)
Tell us the version of Cirq where this happens
1.6.0.dev0 at 1c0ba5b