style: ruff 0.12
Metadata
Current evaluation
Merged after two reviewer approvals. Updates codebase for ruff 0.12 compatibility. CI passed except a non-blocking OSV-scanner failure. Changes affect 8 files with 56 additions and 23 deletions.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [ ] Have you successfully run `make test`?
---
Changes for ruff 0.12.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after two reviewer approvals. Updates codebase for ruff 0.12 compatibility. CI passed except a non-blocking OSV-scanner failure. Changes affect 8 files with 56 additions and 23 deletions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged maintainer PR updating the codebase to ruff 0.12. Changes passed lint checks and were integrated into the main branch without additional review comments. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged maintainer mr-cal's update for ruff 0.12 compatibility. Changes passed lint checks and were integrated into the main branch. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5343 style: ruff 0.11 fixes | snapcraft | merged | Merged minor style updates to align with ruff 0.10 and 0.11. Approved by two reviewers, passed CI checks, and merged with minimal changes across one file. | |
| #803 style: fix linting issues with ruff 0.12 | craft-application | merged | Merged after approval by two reviewers. Resolves linting failures introduced by ruff 0.12 across 14 files. CI passed for fast tests, linting, and security scans, while slow test failures were tracked separately in #804. | |
| #408 style: fix linting for ruff 0.12 | starbase | merged | Merged a one-line update to resolve linting errors for ruff 0.12. Approved by two reviewers and passed all CI checks before integration. | |
| #2383 style: update to ruff 0.12 | charmcraft | merged | Merged after two approvals. Updated project to ruff 0.12 across nine files. Proceeded despite failing CI checks on older Ubuntu snap tests and snap-store tests. | |
| #1156 style: fix linting errors for ruff 0.12.0 | craft-parts | merged | Merged after receiving approval from two reviewers and passing all CI checks. The update resolves linting errors introduced by ruff 0.12.0 across five files, adjusting code formatting to comply with the new linter version. | |
| #969 style: fix linting issues for ruff 0.9 | craft-parts | merged | Merged a minor style fix addressing linting issues for ruff 0.9. Approved by two reviewers, passed CI checks, and updated two files with a net change of one line. | |
| #901 style: fix ruff 0.13 linting issues | craft-application | merged | Merged after passing all CI checks and receiving approval from two reviewers. Resolves ruff 0.13 linting issues with a single-line change to one file. | |
| #6378 style: ruff 0.16 | snapcraft | merged | Merged housekeeping updates: ruff 0.16 configuration, cryptography CVE fix, and removal of an outdated maturin constraint. Approved by two reviewers and passed CI checks prior to merge. | |
| #580 style(ruff): fix linting issues from ruff update | craft-application | merged | Merged after approval by three reviewers. Resolves linting errors from a ruff update, modifying three files with a +7/-7 line change. Successfully integrated into the codebase. | |
| #179 build(deps): update ruff to 0.4.4 | craft-store | merged | Merged update to ruff version 0.4.4. Approved by two reviewers and passed CI checks. The change modified three files with a net addition of five lines. |