← Back to issue list

build(lint): use ruff from snap

View original Github issue

Metadata

Project
snapcraft
Number
#4602
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #4602 (8a9a5c212) replaced pyright-based CI linting with Ruff supplied by its snap for more predictable tool upgrades.

Resolution: closed resolved

Reason: PR #4602 was approved and merged as commit 8a9a5c212, updating linting to use Ruff from its snap and avoiding CI breakage caused by pyright release upgrades.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [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 `tox run -m lint`? - [x] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`) -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #4602 (8a9a5c212) replaced pyright-based CI linting with Ruff supplied by its snap for more predictable tool upgrades.
stealth/space-bunny-alpha — closed resolved Merged PR #4602 replaced pyright-based linting with Ruff supplied by its snap for more stable tool versions; merged as commit 8a9a5c212.
qwen/qwen3.6-35b-a3b — — Merged. Switched CI linting from pyright to ruff installed via snap to ensure toolchain stability. Approved by two reviewers, passed all checks, and updated five files with a +7/-3 line change.
qwen3.6-35b-a3b-mtp-q6 — — Merged. Replaced pyright with Ruff installed via snap to stabilize the CI pipeline, as Ruff's versioning policy prevents unexpected breakages from upstream updates. The change passed all lint and test checks with stable coverage.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#206 build: use ruff from snap craft-application merged Merged as commit 9f8fcc62; four build files were updated to invoke Ruff from its Snap.
83%
#675 style(lint): replace pylint with ruff rockcraft merged Merged as b7d70598, replacing pylint with Ruff, adding Ruff to CI and project requirements, and enabling equivalent lint coverage.
82%
#207 build(deps): use ruff from snap craft-store merged Merged PR #207, commit 27bc107, updating four build files to install and use Ruff from its Snap.
81%
#597 build(deps): use the ruff snap craft-providers merged Merged PR #597 as commit c5f35a0, replacing a frozen pip-installed Ruff version with the latest Ruff snap for development tooling.
80%
#29 style(lint): fix ruff 0.6.0 linting errors craft-platforms merged Merged PR #29 as commit 399c4e0, fixing Ruff 0.6.0 lint errors and switching the pre-commit Ruff hook to rolling versions.
77%
#4939 build(deps): use the ruff snap snapcraft merged Merged in snapcraft#4939 via commit 8f2afd58a, formalizing Ruff snap use, adopting ruff check, and updating compatibility and documentation.
77%
#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.
76%
#624 style(lint): fix ruff 0.6.0 linting errors craft-providers merged Merged as commit 823622d, fixing Ruff 0.6.0 lint errors and switching the pre-commit Ruff hook to rolling versions.
76%
#1387 build: use ruff snap charmcraft merged Merged PR #1387 in commit 4fae04dc, switching Charmcraft CI and tox from Ruff's Python package to the Ruff Snap as a canary.
76%
#423 style(lint): fix ruff 0.6.0 linting errors craft-application merged Merged PR #423 as commit 2f56f14b, applying ruff check --fix across 21 files (+70/-70) to clear the new Ruff 0.6.0 lint errors. Approved by 2 maintainer reviewers.
76%