Freeze ruff version
Metadata
Current evaluation
Merged #4016 as 5b8ad3d08, pinning Ruff in pre-commit and development requirements so upstream lint changes arrive through Renovate PRs.
Resolution: closed resolved
Reason: Pull request #4016 was merged as commit 5b8ad3d08. It pinned Ruff consistently so upstream lint breakages are handled in Renovate update PRs rather than unrelated changes.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Ruff is currently iterating quickly and adding many tests in each new version. This change will move ruff breakages into the renovate update PRs rather than random PRs.
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [x] Have you successfully run `pytest tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #4016 as 5b8ad3d08, pinning Ruff in pre-commit and development requirements so upstream lint changes arrive through Renovate PRs. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #4016 as commit 5b8ad3d08, freezing Ruff so rapid upstream lint changes are handled in Renovate PRs; development requirements were updated to match. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Freezes ruff to prevent CI breakages from rapid updates, routing changes via Renovate. Corrected an oversight in requirements-devel.txt during review. Approved and passed CI. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to freeze the ruff version, preventing unexpected breakages from rapid upstream updates. The maintainer addressed an oversight regarding requirements-devel.txt before merging. Code coverage remains unchanged. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #21 lint: specify a single ruff version | starbase | merged | Merged PR #21 as commit 0f3d7d0, pinning Ruff to one version so updates require Renovate or an explicit PR, avoiding unexpected CI breakage. | |
| #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. | |
| #11 lint: Specify a single ruff version | craft-archives | closed | Superseded by the matching Ruff pin in commit 0f3d7d0 (recorded with #21), restricting updates to Renovate or explicit PRs. | |
| #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. | |
| #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. | |
| #358 lint: specify a single ruff version | craft-parts | merged | Merged PR #358 pinned the Ruff pre-commit hook to one version, restricting updates to Renovate or explicit PRs to prevent unintended CI breakage. | |
| #68 lint: update ruff and fix linting | craft-archives | merged | Merged PR #68 as commit 8bb81ae, updating Ruff and fixing lint violations across four files after prerequisite #67. | |
| #20 style: fix ruff issue | craft-platforms | merged | Merged PR #20 fixed a Ruff linting issue in one file (+1/-3); resolution is commit 71a165d. | |
| #113 update ruff pre-commit | craft-platforms | merged | Merged #113, updating Ruff's pre-commit configuration; the change landed in commit 1740611. | |
| #33 lint: fix lint issues with newer ruff | craft-archives | merged | PR #33 merged as commit c2a6687, fixing new lint failures introduced by a newer Ruff release across3 files (+5/-3). Maintainer deferred extra unit tests to CRAFT-1641. |