pylint: uprev pylint and address new consider-using-with warnings
Metadata
Current evaluation
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.
Resolution: closed resolved
Reason: The pull request was approved by two reviewers and merged into main via merge commit 17c1454, containing commit 53496cf ('requirements-dev: uprev pylint to 2.8.2') and 6d9a8c7 ('multipass, tests: address new consider-using-with warnings with Popen'). Maintainer noted it depended on #40 for other requirement updates, which was also merged.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | 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. | |
| qwen/qwen3.6-35b-a3b | — | closed resolved | Merged PR updating pylint and fixing new `consider-using-with` lint warnings across 5 files. Depends on #40. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
canonical/craft-providers#40
(confidence 90%)
Maintainer comment: 'Depends on #40 for other req updates'; #40 (docs: update and fix doc generation) was merged and re-froze requirements.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #57 tests: update pylint and address new warnings | craft-parts | merged | Merged PR #57 in commit 78560246 upgraded pylint to 2.8.2 and addressed the resulting warnings with minor test changes. | |
| #66 lint: update pylint configuration | craft-store | merged | Merged PR #66 as commit 20c7aa9, refreshing the pylint configuration without functional code changes. |