chore(lint): use ruff from snap
Metadata
Current evaluation
Merged PR canonical/craft-parts#715 (commit e0ecddb2, +3/-2 across 3 files): applied craft-cli#250 so linting installs ruff via `sudo snap install ruff` instead of a uv/pip dependency.
Resolution: closed resolved
Reason: The pull request itself was merged into craft-parts main as commit e0ecddb2 "chore(lint): use ruff from snap (#715)", carrying the same change as canonical/craft-cli#250 (commit 35b6ab3). It was approved by two reviewers with no unresolved comments, and common.mk's install-ruff target now falls back to the ruff snap when ruff is not on PATH.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR canonical/craft-parts#715 (commit e0ecddb2, +3/-2 across 3 files): applied craft-cli#250 so linting installs ruff via `sudo snap install ruff` instead of a uv/pip dependency. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged canonical/craft-cli#250, switching linting from a Ruff dependency to the Ruff snap across three files (+5/-5). The change received two approvals. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a linting chore switching to ruff from snap. Approved by two reviewers, the change modifies three files with a net reduction of two lines. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the linting configuration to use ruff installed via snap. The update replaces previous linting dependencies with the snap package to ensure consistent tooling across development environments. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/craft-cli#250
(confidence 92%)
Original "chore: switch to ruff snap" PR that craft-parts#715 merged back into its own tree (same 3-file ruff snap change).
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #4417 chore(deps): update ruff to 0.1.1 and fix lint issues | snapcraft | merged | Ruff 0.1.1 dependency bump plus lint fixes merged to snapcraft main as commit 4b1cff546 (PR #4417, +66/-36 across 19 files). A later follow-up, style: ruff 0.16 (#582), superseded the pin. | |
| #356 chore(deps): update dependency lint/ruff to v0.0.277 | craft-providers | merged | Merged in canonical/craft-providers#356 via commit bb4ff5a, updating lint/ruff from 0.0.272 to 0.0.277. | |
| #205 chore(deps): update dependency lint/ruff to v0.2.1 - autoclosed | craft-application | closed | Renovate PR #205 was auto-closed unmerged; PR #206 replaced its Ruff package pin with Ruff from the Snap. | |
| #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. | |
| #137 chore(deps): update dependency lint/ruff to v0.1.7 | craft-store | merged | Merged canonical/craft-store#137 in commit d210aff, upgrading lint/ruff from v0.1.1 to v0.1.7. | |
| #252 chore(deps): update dependency lint/ruff to ~=0.4.1 - autoclosed | craft-cli | closed | Renovate PR bumping lint/ruff to ~=0.4.1 was autoclosed unmerged: the ruff dependency had already been dropped in favour of the ruff snap by PR #250 (commit 35b6ab3), making the version bump obsolete. | |
| #595 chore(deps): update dependency lint/ruff to v0.5.1 | craft-providers | closed | Renovate PR to bump lint/ruff 0.4.10 → 0.5.1 was closed unmerged; maintainer @lengau closed it as "superseded by #597", which replaced the pinned ruff dep with the ruff snap. | |
| #524 chore(deps): update dependency lint/ruff to v0.3.0 | craft-providers | merged | Merged the Ruff lint dependency upgrade from 0.1.7 to 0.3.0 in canonical/craft-providers PR #524. | |
| #204 chore: use ruff snap | starbase | merged | Merged PR #204 in commit 09b91ba, switching Ruff usage in four files to the Ruff snap. | |
| #250 chore: switch to ruff snap | craft-cli | merged | Merged PR #250 as commit 35b6ab3, switching Ruff usage to its snap across three files (+5/-5); it received two approvals. |