style: remove pylint
Metadata
Current evaluation
Merged removal of pylint configuration, fully replaced by ruff. Approved by three reviewers, passed CI checks, and modified 46 files across the codebase.
Suggested action: —
No scores available.
Issue body
Pylint is pretty much entirely replaced with ruff at this point.
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox run -m lint`?
- [x] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged removal of pylint configuration, fully replaced by ruff. Approved by three reviewers, passed CI checks, and modified 46 files across the codebase. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Pylint removed and replaced with ruff for linting. The change passed all lint and test checks, then merged successfully by maintainer lengau. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the pull request to remove pylint, which has been fully replaced by ruff. The change was successfully integrated into the main branch. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #118 style: remove pylint | craft-store | merged | Merged to remove pylint from the project. Approved by one reviewer, passed CI checks, and modified two files with a net reduction of three lines. | |
| #562 Remove pylint in favour of ruff | craft-parts | closed | 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. | |
| #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. | |
| #4848 style: remove pylint comments | snapcraft | merged | Merged cleanup removing remaining pylint comments. Approved by two reviewers, passed CI checks, and updated two files with minimal changes. | |
| #2229 style: fix linting errors | charmcraft | merged | Merged changes resolving linting errors from the latest ruff and pyright. Approved by two reviewers with zero unresolved comments. Updated 8 files, removing 22 lines overall. | |
| #291 lint: remove leftover pylint directive | craft-parts | merged | Merged a change removing a leftover pylint directive. The single-line deletion across one file received reviewer approval and was integrated after passing checks. | |
| #66 lint: update pylint configuration | craft-store | merged | Merged an update to the pylint configuration, removing 627 lines of outdated settings and adding 16 lines with no functional changes. Approved by a reviewer and passed all CI checks. | |
| #139 style: stricter ruff linters | starbase | merged | Merged stricter ruff linter rules across six files (+50/-33 lines). Approved by four reviewers, passed CI checks, and maintained 100% test coverage. | |
| #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. | |
| #319 style: update linting configuration for ruff 0.9 | starbase | merged | Merged update to linting configuration for ruff 0.9. Approved by one reviewer and passed CI. The minimal two-file change was accepted after the author deferred an unrelated git_describe_command suggestion to a separate PR. |