tests: update pylint and address new warnings
Metadata
Current evaluation
Merged PR #57 in commit 78560246 upgraded pylint to 2.8.2 and addressed the resulting warnings with minor test changes.
Resolution: closed resolved
Reason: PR #57 was approved and merged as commit 78560246, with implementation commit 65c2bfd7 updating pylint and fixing the new warnings.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Pylint 2.8.2 adds more warnings, address them.
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [x] 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 #57 in commit 78560246 upgraded pylint to 2.8.2 and addressed the resulting warnings with minor test changes. | |
| stealth/space-bunny-alpha | — | closed resolved | PR #57 merged: bumped test/lint deps to pylint 2.8.2 and fixed the new warnings in the test suite (commit 65c2bfd7, merge 78560246). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #57 updated pylint to 2.8.2 and addressed the resulting warnings with minor test-suite changes in one file. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged updates to pylint 2.8.2 and resolved newly triggered warnings in the test suite. The maintainer addressed the linter changes and the pull request was approved and merged. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #29 tests: address pylint issues | craft-parts | merged | Merged: small test-only cleanup of Pylint warnings (3 files, +7/-4) by maintainer cmatsuoka. Landed as commit 0a3d2554 via merge commit efb9bcc3 (PR #29). | |
| #41 pylint: uprev pylint and address new consider-using-with warnings | craft-providers | merged | PR #41 merged (commit 17c1454): bumped pylint to 2.8.2 (53496cf) and refactored code (6d9a8c7) to silence new consider-using-with warnings; landed after dependency PR #40. | |
| #66 lint: update pylint configuration | craft-store | merged | Merged PR #66 as commit 20c7aa9, refreshing the pylint configuration without functional code changes. | |
| #42 pylint shows some warnings | craft-cli | closed | Pylint warnings were fixed and pylint was enforced in GitHub CI by merged PR #59 (commit ad06d5c; merge 6dafc86). | |
| #11 tests: configure pylint to issue fixme-info | craft-parts | merged | Merged #11 via commit 08b51a7c, configuring pylint-fixme-info to report TODO/FIXME messages as informational and removing pylint's --fail-under CI check. | |
| #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. | |
| #3813 lint: bump pylint | snapcraft | merged | Merged in commit 522b301a9: updated Pylint and removed a redundant too-many-lines suppression across three files. |