← Back to issue list

build: use ruff snap

View original Github issue

Metadata

Project
charmcraft
Number
#1387
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged. Replaces ruff usage in CI and tox with the snap package as a canary test before broader rollout. Approved by two reviewers.

Suggested action:

No scores available.

Issue body

Uses the version of ruff from snap in CI and tox. Testing this in charmcraft as a canary before changing all of our repos over.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. Replaces ruff usage in CI and tox with the snap package as a canary test before broader rollout. Approved by two reviewers.
qwen3.6-35b-a3b-mtp-q6 Merged. Updates CI and tox to use the ruff snap package, serving as a canary test for broader repository adoption.
qwen3.6-35b-a3b-mtp-q6 Merged. Configures CI and tox to use the ruff snap package as a canary test before wider repository adoption.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#206 build: use ruff from snap craft-application merged Merged to update the build configuration to use ruff from snap. Approved by two reviewers, passed CI checks, and applied minimal changes across four files.
85%
#4939 build(deps): use the ruff snap snapcraft merged Merged updates to the CI workflow to use the ruff snap. Adds the ruff check command, updates compatibility for ruff 0.5.4, and includes documentation changes. Approved and passed all checks.
82%
#207 build(deps): use ruff from snap craft-store merged Merged update to use ruff from snap for build dependencies. Approved by two reviewers with six lines added and four removed across four files. No CI failures reported.
80%
#4602 build(lint): use ruff from snap snapcraft merged 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.
78%
#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.
73%
#165 style: fix linting for ruff 0.12 imagecraft merged Merged a pre-emptive linting update for ruff 0.12 compatibility. Approved by two reviewers and passed CI. The change adjusts formatting across four files to prevent breaking changes when the snap reaches stable.
72%