← Back to issue list

style: remove pylint

View original Github issue

Metadata

Project
craft-store
Number
#118
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #118 as commit beb9201, removing Pylint configuration and tooling after Ruff had replaced it.

Resolution: closed resolved

Reason: The cleanup was merged into craft-store as commit beb9201, completing the removal of Pylint after its replacement by Ruff.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #118 as commit beb9201, removing Pylint configuration and tooling after Ruff had replaced it.
stealth/space-bunny-alpha — closed resolved Merged as canonical/snapcraft#4817, removing pylint configuration after it was fully replaced by Ruff.
qwen/qwen3.6-35b-a3b — — 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.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/snapcraft#4817 (confidence 65%)

    Upstream Snapcraft PR performed the equivalent Pylint cleanup after Ruff replaced it.

Related issues

Issue Project State Summary Similarity
#4817 style: remove pylint snapcraft merged Merged as a9badaa32, removing Pylint from lint tooling, configuration, and source suppressions after Ruff had replaced nearly all of it.
90%
#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.
85%
#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.
78%
#291 lint: remove leftover pylint directive craft-parts merged Merged PR #291 in commit fb583a60, removing one leftover pylint directive from a single file.
76%
#66 lint: update pylint configuration craft-store merged Merged PR #66 as commit 20c7aa9, refreshing the pylint configuration without functional code changes.
76%
#565 ci: pin pylint version craft-parts merged Merged PR #565, commit 174d0e88, pinned pylint in CI for stability; migration to Ruff was deferred.
75%
#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.
75%
#481 build: update ruff settings rockcraft closed Ruff lint-config tweak PR closed unmerged; maintainer lengau closed it 2024-08-28 as superseded by #675 ("style(lint): replace pylint with ruff", merged as commit b7d70598).
74%
#59 Run pylint in Github CI. craft-cli merged Merged PR #59 added pylint to GitHub CI via ad06d5c (merge 6dafc86); later PR #173 replaced pylint with Ruff.
73%
#173 tools: enable ruff craft-cli merged Merged PR #173 enabled Ruff, removed pylint, and fixed related mypy, pyright, test, and coverage issues; it explicitly fixes canonical/craft-cli#146.
73%