← Back to issue list

lint: remove leftover pylint directive

View original Github issue

Metadata

Project
craft-parts
Number
#291
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

Merged a change removing a leftover pylint directive. The single-line deletion across one file received reviewer approval and was integrated after passing checks.

Suggested action:

No scores available.

Issue body

- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a change removing a leftover pylint directive. The single-line deletion across one file received reviewer approval and was integrated after passing checks.
qwen3.6-35b-a3b-mtp-q6 Merged by maintainer tigarmo, this pull request removes a leftover pylint directive to resolve a linting issue.

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.
84%
#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.
82%
#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.
79%
#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.
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.
78%
#33 Linting imagecraft merged Merged a one-line linting fix from a maintainer. The change modified a single file with one addition and one deletion, requiring no reviews or CI checks before integration.
77%
#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.
76%
#36 Lint imagecraft merged Merged a single-file linting adjustment with one addition and one deletion. The maintainer-submitted change required no reviews or CI checks before integration.
76%
#164 lint: compliance with pylint-2.12.2 (CRAFT-716) craft-parts merged Merged to enforce pylint 2.12.2 compliance. Approved by two reviewers. One reviewer suggested disabling a default encoding warning instead of explicitly setting it. The branch was merged with 15 unresolved comments.
76%