← Back to issue list

lint: update pylint configuration

View original Github issue

Metadata

Project
craft-store
Number
#66
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

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.

Suggested action:

No scores available.

Issue body

The pylint configuration got out of date. Update with no functional changes. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged an update to the outdated pylint configuration. The change introduces no functional modifications and was signed off and merged by maintainer sergiusens to align project linting rules.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3813 lint: bump pylint snapcraft merged Merged update bumping pylint to the latest version and removing the redundant disable: too-many-lines configuration. Approved, passed CI, and integrated into main as part of core22 architecture support.
86%
#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.
82%
#57 tests: update pylint and address new warnings craft-parts merged Merged after two approvals and passing CI. Updated pylint to 2.8.2 and resolved newly introduced linting warnings in the test suite with minor code adjustments across a single file.
80%
#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.
79%
#166 linters: re-enable pylint fixme info craft-parts merged Merged a one-line change to re-enable pylint fixme info in the linters configuration. Approved by one reviewer, passed all CI checks, and resolved with zero outstanding comments.
78%
#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.
78%
#3747 cli: disable new pylint false positive snapcraft merged Merged a change to disable a new pylint false positive in the CLI. Approved by a reviewer, passed CI and lint checks, and modified one file. The author confirmed unrelated spread failures did not impact the change.
77%
#77 lint: fix linting issues with up-to-date linters craft-store merged Merged a maintainer-submitted pull request resolving linting errors from updated linter versions. Approved by one reviewer, passed CI, and modified 11 files (+40/-24 lines).
77%
#4073 lint: reintroduce pylint in tox snapcraft merged Merged into main following approval and passing CI checks. The change reintroduces pylint to the tox configuration, adding 15 lines and removing 1 in a single file. Code coverage remains unchanged.
76%
#90 many: address new linter warnings (CRAFT-364) craft-parts merged Merged after resolving new linter warnings from updated mypy and pylint versions, including silencing checks in tests. Approved by two reviewers, passed CI, and modified eight files (+32/-22 lines).
76%