style: fix linting for ruff 0.12
Metadata
Current evaluation
Merged #165, incorporating canonical/starbase#408, with a four-file Ruff 0.12 compatibility style update (+7/-7).
Resolution: closed resolved
Reason: PR #165 was merged after two approvals, carrying canonical/starbase#408 and applying the Ruff 0.12 lint fixes. The author described it as preemptive against Ruff 0.12.0's potentially breaking changes.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint && make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #165, incorporating canonical/starbase#408, with a four-file Ruff 0.12 compatibility style update (+7/-7). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged starbase#408 with a four-file, +7/-7 style update to make linting compatible with Ruff 0.12, ahead of Ruff’s potentially breaking release. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a pre-emptive linting update for ruff 0.12 compatibility. Approved by two reviewers and passed CI. The change adjusts formatting across four files to prevent breaking changes when the snap reaches stable. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to align linting rules with ruff 0.12.0. Implemented pre-emptively by the maintainer ahead of the snap stable release to prevent breaking changes flagged by the community. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/starbase#408
(confidence 99%)
Merged Starbase PR containing the corresponding Ruff 0.12 style fix incorporated into Imagecraft PR #165.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #408 style: fix linting for ruff 0.12 | starbase | merged | Merged PR #408 as commit dc41fe4, applying a one-line style change that restored linting compatibility with Ruff 0.12. | |
| #1737 style: fix linting for ruff 0.5 | charmcraft | merged | Merged PR #1737 as commit 8994523e, updating one file (+2/-6) to fix linting under Ruff 0.5. | |
| #1156 style: fix linting errors for ruff 0.12.0 | craft-parts | merged | Ruff 0.12.0 lint fixes merged as PR #1156 (commit eb82b792), correcting style violations in 5 files (+9/-12) after the ruff-pre-commit v0.12.0 bump in #407; all CI checks passed. | |
| #357 style: fix lints from ruff 0.12 | craft-cli | merged | Merged as dd635c9, fixing new Ruff 0.12 lint errors across seven files (+11/-10); all CI checks passed. | |
| #1465 style: fix linting issues for newer ruff | craft-parts | merged | Merged #1465 as a58695ef, fixing five files (+13/-6) for compatibility with newer Ruff; all CI checks passed. | |
| #173 style: fix lint for ruff 0.12 | craft-archives | merged | Merged PR #173 as commit 53d732e, fixing lint for Ruff 0.12 while retaining Ruff 0.11 compatibility; all CI checks passed. | |
| #803 style: fix linting issues with ruff 0.12 | craft-application | merged | Merged via commit 5e15ff09, fixing Ruff 0.12 lint failures across 14 files. A separate integration-test failure was fixed by #804. | |
| #5574 style: ruff 0.12 | snapcraft | merged | Merged Ruff 0.12 compatibility and style updates in commit 94a594fd3 after approval by two reviewers. | |
| #969 style: fix linting issues for ruff 0.9 | craft-parts | merged | Merged as commit f1427855: small Ruff 0.9 lint fixes (+2/-1) across 2 files, following the ruff-pre-commit hook bump to the 0.9.x series. | |
| #580 style(ruff): fix linting issues from ruff update | craft-application | merged | Merged PR canonical/craft-application#580 (commit 97347705) by lengau applied style-only fixes (+7/-7, 3 files) for new lint failures caused by a Ruff update; approved by 3 reviewers. |