tests: update pylint and address new warnings
Metadata
Current evaluation
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.
Suggested action: —
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 |
|---|---|---|---|---|
| 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 to address pylint warnings in test files. Approved by three reviewers and passed CI. The update modified three files, adding seven and removing four lines. | |
| #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. | |
| #90 many: address new linter warnings (CRAFT-364) | craft-parts | merged | Merged after resolving new linter warnings from updated mypy and pylint versions, including silencing checks in tests. Approved by two reviewers, passed CI, and modified eight files (+32/-22 lines). | |
| #11 tests: configure pylint to issue fixme-info | craft-parts | merged | Merged. Configured pylint to treat TODO and FIXME messages as informational instead of warnings or errors, removing the --fail-under CI flag. Approved by two reviewers and passed all checks. | |
| #856 style: update mypy and fix linting issues | craft-application | merged | Merged after receiving approval from two reviewers and passing all CI checks. The update refreshed mypy and resolved linting issues across two files, resulting in 38 additions and 32 deletions. | |
| #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. | |
| #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. | |
| #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. | |
| #3672 linters: address issues and enable pylint in ci | snapcraft | merged | Merged after approval and CI pass. The PR addressed linter issues and enabled pylint in CI across 8 files. Changes increased test coverage by 0.02% with 100% diff coverage, successfully integrating static analysis into the pipeline. | |
| #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. |