chore(deps): update ruff and fix linting issue
Metadata
Current evaluation
Routine housekeeping PR bumping the ruff pre-commit hook and fixing the resulting lint error (2 files, +2/-4). Approved by 2 reviewers and merged into main as commit 59ab0049; Codecov's -0.10% coverage note was non-blocking.
Resolution: closed resolved
Reason: The PR itself was merged into main as commit 59ab0049 ("chore(deps): update ruff and fix linting issue") with review status approved by 2 reviewers and zero unresolved review comments. The Ruff version bump and accompanying lint fix are both present in the main branch history.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `tox`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Routine housekeeping PR bumping the ruff pre-commit hook and fixing the resulting lint error (2 files, +2/-4). Approved by 2 reviewers and merged into main as commit 59ab0049; Codecov's -0.10% coverage note was non-blocking. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged into main via #115 after two approvals, updating Ruff and fixing a linting issue across two files. Codecov reported a 0.10% coverage decrease. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged into main after two approvals and passing CI. The update refreshed the ruff dependency and resolved a linting issue across two files. Codecov recorded a negligible 0.10% coverage decrease upon merge. |
Update history
No update history recorded yet.
Related work
-
Related To:
craft-application#177
(confidence 40%)
Later Dependabot commit c6d29c32 bumped the ruff dependency to v0.1.15, continuing the Ruff upgrade track this PR advanced.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1347 chore(deps): update ruff and fix new linting errors | charmcraft | merged | Ruff pre-commit hook was bumped and resulting lint errors auto-fixed across 10 files (+40/-37). PR #1347 was merged by lengau on 2023-10-23 as commit 34018724. | |
| #119 chore(deps): update dependency lint/ruff to v0.0.284 | starbase | merged | Renovate dependency bump PR #119 (lint/ruff 0.0.272 -> 0.0.284 in the ruff pre-commit hook) was approved and merged into main as commit be3ae62; +1/-1 line, no source changes. | |
| #230 chore(deps): update dependency lint/ruff to v0.1.15 | craft-cli | merged | Renovate PR merged (commit e68b0c7), bumping the ruff lint hook from 0.1.14 to 0.1.15 in .pre-commit-config.yaml; approved by two reviewers with no CI failures. | |
| #130 chore(deps): update dependency lint/ruff to v0.0.290 | starbase | merged | Renovate PR merged into main via commit 4e97b3f, bumping the pinned lint/ruff from 0.0.285 to 0.0.290 (one-line change, no coverage impact). | |
| #578 chore(deps): update dependency lint/ruff to v0.4.10 | craft-providers | merged | Renovate PR merged: bumped the lint/ruff dependency from 0.4.8 to 0.4.10 (commit 117db56) after reviewer approval; a single-line version change. | |
| #36 chore: switch to ruff | craft-platforms | merged | Merged PR #36 as commit ce5f6b9, switching linting and formatting to Ruff, updating configuration, and applying fixes across seven files. | |
| #181 chore(deps): update dependency lint/ruff to v0.1.14 | craft-application | merged | Renovate PR bumping lint/ruff from ==0.1.11 to ==0.1.14 was approved by 3 reviewers and merged into main (commit bf491e38), a one-line pyproject.toml change; CI and Codecov checks were clean. | |
| #217 chore(deps): update dependency lint/ruff to v0.1.8 | craft-cli | merged | Renovate PR bumping lint/ruff 0.1.7 to 0.1.8 was approved by two reviewers and merged into main as commit 61f7791 (Dec 2023); later superseded by ruff 0.1.14 (#223) and 0.1.15 (#230). | |
| #566 chore(deps): update dependency lint/ruff to v0.4.3 | craft-providers | merged | Renovate PR #566 bumped the lint/ruff pin from ==0.4.2 to ==0.4.3 (+1/-1 line). Approved by @mr-cal (who noted ruff would move to a snap at the next Starbase merge) and merged as commit e7b2ddc. | |
| #190 chore(deps): update dependency lint/ruff to v0.2.0 | craft-application | merged | Renovate upgraded lint/ruff from 0.1.14 to 0.2.0; PR #190 was approved and merged in commit 9d7306ca. |