lint: remove leftover pylint directive
Metadata
Current evaluation
Merged PR #291 in commit fb583a60, removing one leftover pylint directive from a single file.
Resolution: closed resolved
Reason: PR #291 was approved with no unresolved review comments and merged as commit fb583a60. The one-line deletion completed the pylint-directive cleanup.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #291 in commit fb583a60, removing one leftover pylint directive from a single file. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged canonical/craft-parts#291 (commit fb583a60), removing one leftover pylint directive from a single file. | |
| 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 |
|---|---|---|---|---|
| #216 lint: remove pylint useless suppression "too-many-lines" | craft-providers | merged | Merged in a8d5fc5: removed the obsolete pylint too-many-lines suppression after the file fell below 1,000 LOC. | |
| #3813 lint: bump pylint | snapcraft | merged | Merged in commit 522b301a9: updated Pylint and removed a redundant too-many-lines suppression across three files. | |
| #66 lint: update pylint configuration | craft-store | merged | Merged PR #66 as commit 20c7aa9, refreshing the pylint configuration without functional code changes. | |
| #118 style: remove pylint | craft-store | merged | Merged PR #118 as commit beb9201, removing Pylint configuration and tooling after Ruff had replaced it. | |
| #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. | |
| #33 Linting | imagecraft | merged | Merged the one-line linting cleanup from PR #33 as commit aed3b2d via merge commit 44d5d39. | |
| #164 lint: compliance with pylint-2.12.2 (CRAFT-716) | craft-parts | merged | Merged PR #164 as 53ee6b27, updating 41 files for pylint 2.12.2 compliance; its explicit file-I/O encoding changes were later reverted in 42d91e33. | |
| #192 lint: enable pylint useless-supression, minor reformatting | craft-providers | merged | Merged lint cleanup in craft-providers#192 via commit f644c25, enabling pylint's useless-suppression check, removing obsolete suppressions, and reformatting seven files without functional changes. | |
| #3747 cli: disable new pylint false positive | snapcraft | merged | 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. | |
| #4848 style: remove pylint comments | snapcraft | merged | Merged as snapcraft#4848 (commit f5e002565): a small two-file style cleanup deleting the last remaining pylint comments left over after #4817, approved by two reviewers. |