lint: compliance with pylint-2.12.2 (CRAFT-716)
Metadata
Current evaluation
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.
Resolution: closed resolved
Reason: The maintainer-approved PR was merged in commit 53ee6b27. Although its explicit file-I/O encoding changes were subsequently reverted in commit 42d91e33, the pylint 2.12.2 compliance changes were merged.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [X] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
lint: compliance with pylint-2.12.2
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | 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. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #164 as 53ee6b27, updating 41 files for pylint 2.12.2 compliance. Two reviewers approved it despite 15 unresolved review comments. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to enforce pylint 2.12.2 compliance. The update addresses linting rules and incorporates feedback regarding explicit default encoding handling versus disabling specific warnings. Changes were accepted and integrated. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3813 lint: bump pylint | snapcraft | merged | Merged in commit 522b301a9: updated Pylint and removed a redundant too-many-lines suppression across three files. | |
| #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. | |
| #127 lint: update pylint to 2.14 | craft-providers | merged | Merged PR #127 via commit a38cb5a, updating .pylintrc for pylint 2.14 compatibility in change 69a2c0e. | |
| #66 lint: update pylint configuration | craft-store | merged | Merged PR #66 as commit 20c7aa9, refreshing the pylint configuration without functional code changes. | |
| #291 lint: remove leftover pylint directive | craft-parts | merged | Merged PR #291 in commit fb583a60, removing one leftover pylint directive from a single file. | |
| #18 lint: update linters | craft-archives | merged | Merged in canonical/craft-archives#18 via commit 41edc5b, updating linters across two files. |