← Back to issue list

Remove pylint in favour of ruff

View original Github issue

Metadata

Project
craft-parts
Number
#562
Type
issue
State
closed
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Resolved by migrating from pylint to ruff to fix CI breaks caused by pylint 3.0. Commit 111c8907 removed pylint configuration and dependencies, completing the switch via PR #580.

Suggested action: —

No scores available.

Issue body

### What needs to get done 1. Ensure that all of our enabled pylint checks are also enabled in the ruff configuration 2. Remove pylint from CI 3. Remove pylint from dependencies 4. Remove pylint from Makefile 5. Disable ruff linters that require code changes ### Why it needs to get done Pylint 3.0 has caused our CI to break. https://github.com/canonical/craft-parts/actions/runs/6436125741/job/17478834266?pr=547 From internal discussion we've decided to remove it in favour of ruff rather than fixing it

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Resolved by migrating from pylint to ruff to fix CI breaks caused by pylint 3.0. Commit 111c8907 removed pylint configuration and dependencies, completing the switch via PR #580.
qwen/qwen3.6-35b-a3b
Staleness: 95
Complexity: 15
Confidence: 85
Support Request: 0
needs triage Task to migrate project linting from pylint to ruff. Open for over two years with no maintainer assessment, labels, or recent activity beyond an automated Jira sync.
qwen3.6-35b-a3b-mtp-q6
Staleness: 90
Complexity: 40
Confidence: 75
Support Request: 5
close stale Task to migrate CI linting from pylint to ruff after pylint 3.0 broke builds. Open for over two years with no maintainer follow-up or implementation progress.
qwen3.6-35b-a3b-mtp-q6
Staleness: 90
Complexity: 15
Confidence: 85
Support Request: 0
close stale Migrate linting from pylint to ruff after pylint 3.0 broke CI. Requires syncing checks, removing pylint from CI, dependencies, and Makefile, and disabling incompatible ruff rules. Open and unassigned with stale activity.

Update history

Date Change
closed

Related issues

Issue Project State Summary Similarity
#4817 style: remove pylint snapcraft merged Merged removal of pylint configuration, fully replaced by ruff. Approved by three reviewers, passed CI checks, and modified 46 files across the codebase.
84%
#675 style(lint): replace pylint with ruff rockcraft merged Merged a pull request replacing pylint with ruff for linting. The author updated CI configuration and requirements files to include ruff and remove pylint, ensuring all linters were properly enabled. Approved by reviewers and passed CI checks.
79%
#173 tools: enable ruff craft-cli merged Merged. Enabled ruff, removed pylint, and resolved associated linting, type-checking, and test coverage issues. Increased overall coverage by 0.08% and resolved issue #146.
73%
#4712 Move more linting to ruff snapcraft closed 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.
72%
#408 style: fix linting for ruff 0.12 starbase merged Merged a one-line update to resolve linting errors for ruff 0.12. Approved by two reviewers and passed all CI checks before integration.
72%
#4012 linters: migrate to ruff snapcraft merged Migrated linters to Ruff with a legacy-specific ruff.toml. Approved by two reviewers, passed CI with 100% diff coverage, and merged into main with 167 additions and 35 deletions across 20 files.
71%
#317 chore: update ruff configuration for 0.8 craft-cli merged Merged. Removed two unsupported linter rules from pyproject.toml to align with ruff 0.8. Approved by two reviewers, passed CI, and merged after auto-format resolved remaining linting errors.
71%
#1156 style: fix linting errors for ruff 0.12.0 craft-parts merged Merged after receiving approval from two reviewers and passing all CI checks. The update resolves linting errors introduced by ruff 0.12.0 across five files, adjusting code formatting to comply with the new linter version.
71%
#173 style: fix lint for ruff 0.12 craft-archives merged Merged following two approvals and passing CI checks. Resolves lint compatibility for ruff 0.12 while preserving backwards compatibility with ruff 0.11. Applied via a minimal two-line adjustment.
71%
#33 lint: fix lint issues with newer ruff craft-archives merged Merged to resolve linting failures introduced by a newer ruff version. Changes fix style checks across three files. Review approved, CI passed, and coverage remained unchanged. Additional unit tests were deferred to a separate tracking ticket.
70%