← Back to issue list

cli: disable new pylint false positive

View original Github issue

Metadata

Project
snapcraft
Number
#3747
Type
pull request
State
merged
Author
sergiusens
Labels
—
Created
Updated
Closed

Current evaluation

PR #3747 merged via 12d7cb60c, adding a targeted CLI suppression for a new pylint false positive; the unrelated spread failure did not affect the change.

Resolution: closed resolved

Reason: The approved PR was merged in commit 574de3de1, with its change committed as 12d7cb60c. This resolved the lint failure by suppressing the erroneous pylint warning.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [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 `make lint`? - [x] Have you successfully run `pytest tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved PR #3747 merged via 12d7cb60c, adding a targeted CLI suppression for a new pylint false positive; the unrelated spread failure did not affect the change.
stealth/space-bunny-alpha — closed resolved Merged in canonical/snapcraft#3747 via commit 12d7cb60c, suppressing a new pylint false positive in the CLI; the unrelated spread failure was noted as non-impacting.
qwen/qwen3.6-35b-a3b — — 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.
qwen3.6-35b-a3b-mtp-q6 — — Merged to disable a newly introduced pylint false positive in the CLI. The maintainer dismissed unrelated spread failures and verified all lint and unit tests passed prior to merging.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#291 lint: remove leftover pylint directive craft-parts merged Merged PR #291 in commit fb583a60, removing one leftover pylint directive from a single file.
70%