Move more linting to ruff
Metadata
Current evaluation
Ruff linting was completed and aligned with Starbase; pylint and isort were removed in #4817 and commit a00f1c439, followed by pylint cleanup in #4848.
Resolution: —
Evaluated by: mr-cal-bot
No scores available.
Issue body
### What needs to get done
1. Bring the latest linting config from starbase as much as possible
2. Replace isort with relevant ruff config. (Any unimplemented config is okay to ignore as long as ruff doesn't rewrite unchanged files)
3. Disable flake8 and pylint tests that are repeated by ruff
### Why it needs to get done
Continuous improvement on the code quality front
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | — | Ruff linting was completed and aligned with Starbase; pylint and isort were removed in #4817 and commit a00f1c439, followed by pylint cleanup in #4848. | |
| stealth/space-bunny-alpha | — | closed resolved | Ruff linting was completed and aligned with Starbase; redundant pylint and isort configurations were removed. | |
| qwen/qwen3.6-35b-a3b | — | — | Completed. Linting configuration was updated to align with starbase, replacing isort with ruff and disabling redundant flake8 and pylint tests. Maintainer confirmed the changes and closed the issue. | |
| qwen/qwen3.6-35b-a3b | — | — | Completed. Linting was migrated to ruff, isort was replaced, and redundant flake8 and pylint tests were disabled. Snapcraft is now aligned with the starbase linting configuration. | |
| qwen/qwen3.6-35b-a3b | — | — | Completed. Linting was migrated to ruff, replacing isort and disabling redundant flake8 and pylint checks. The maintainer confirmed the project is now fully aligned with the starbase linting configuration. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Resolved. Linting configuration was updated to align with starbase, replacing isort with ruff and disabling redundant flake8 and pylint tests. Snapcraft is now up-to-date. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #675 style(lint): replace pylint with ruff | rockcraft | merged | Merged as b7d70598, replacing pylint with Ruff, adding Ruff to CI and project requirements, and enabling equivalent lint coverage. | |
| #55 lint: stricter ruff rules | starbase | merged | Merged PR canonical/starbase#55 as b5cda0d, tightening Ruff lint rules in one file (+23/-9) after maintainer approval. | |
| #562 Remove pylint in favour of ruff | craft-parts | closed | Pylint was dropped in favour of ruff by merged PR #580 (commit 111c8907, "chore(linter): enable ruff base on starbase config"), removing the [tool.pylint] config and CI/Makefile/dependency usage; ruff config expanded to cover it. | |
| #444 fix: resolve ruff and ty lint findings | craft-store | merged | Merged to fix ruff and ty lint findings and update ruff config, preparing for the Starbase sync in #440. | |
| #264 lint: format with ruff and fix ruff linting errors | rockcraft | closed | Ruff formatting and lint cleanup in #264 was closed unmerged after remaining unreviewed. Prerequisite #263 merged as 4abd39c2, while later Ruff adoption continued in #363. | |
| #907 style: adjust linting rules for ruff 0.12 | rockcraft | merged | Merged into main as 186cb0fb, updating Ruff 0.12 settings to ignore PLC0415 and avoid unnecessary noqa comments. | |
| #4981 style(lint): fix ruff 0.6 linting errors | snapcraft | merged | Merged PR #4981 as commit 3a1b8a949, fixing Ruff 0.6 lint errors with a minimal one-file change. | |
| #288 lint: add ruff, remove autoflake, flake8, isort, pydocstyle, pylint | craft-providers | merged | Merged PR #288 in commit 42a7920, replacing autoflake, flake8, isort, pydocstyle, and pylint with Ruff; deferred Ruff rule cleanup was tracked in #287. | |
| #165 style: fix linting for ruff 0.12 | imagecraft | merged | Merged #165, incorporating canonical/starbase#408, with a four-file Ruff 0.12 compatibility style update (+7/-7). | |
| #68 lint: update ruff and fix linting | craft-archives | merged | Merged PR #68 as commit 8bb81ae, updating Ruff and fixing lint violations across four files after prerequisite #67. |