← Back to issue list

chore: update ruff

View original Github issue

Metadata

Project
craft-parts
Number
#700
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged after approval by two reviewers and passing CI checks. Updates ruff configuration across four files, supersedes #699, and resolves a lint issue addressed in #702.

Suggested action:

No scores available.

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- This comes in three commits so the changes are more obvious. Supersedes #699

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after approval by two reviewers and passing CI checks. Updates ruff configuration across four files, supersedes #699, and resolves a lint issue addressed in #702.
qwen3.6-35b-a3b-mtp-q6 Merged after superseding #699. The ruff update was applied across three commits, with a lint issue resolved in #702 prior to integration.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#192 chore: update ruff settings starbase merged Merged updates to ruff configuration settings across two files. Approved by one reviewer, passed CI checks, and integrated into the main branch.
92%
#200 chore(deps): update ruff and its configuration craft-application merged Merged an update to ruff and its configuration. Approved by two reviewers with passing CI checks. The change modified one file with five additions and five deletions.
92%
#1576 chore: update ruff configs charmcraft merged Merged update to Ruff configuration files across seven projects. Changes totaled 35 additions and 35 deletions. Approved by two reviewers with no unresolved comments.
91%
#1347 chore(deps): update ruff and fix new linting errors charmcraft merged Merged after updating the ruff dependency and fixing new linting errors. Modifications were largely autoformatted across ten files. The changes received dual reviewer approval, passed CI checks, and were successfully integrated.
91%
#115 chore(deps): update ruff and fix linting issue craft-application merged 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.
89%
#4417 chore(deps): update ruff to 0.1.1 and fix lint issues snapcraft merged Merged dependency update upgrading ruff to 0.1.1 and resolving linting errors across 19 files. Approved by two reviewers, passed CI, and increased coverage by 0.01%. A newer ruff version was released shortly after merge.
88%
#22 chore(lint): update ruff and config starbase merged Merged ruff linter and configuration updates. Added an ignore rule for test functions with excessive arguments. Approved by one reviewer, modifying two files with a +4/-2 line change.
88%
#146 chore(deps): include ruff in dev updates starbase merged Merged into main after approval and passing CI. The update incorporates the ruff linter into development dependencies following its 0.1 release, changing one file with no effect on test coverage.
86%
#103 chore: fix new lints from ruff 0.12 craft-grammar merged Merged after approval and passing CI checks. Updated pre-commit ruff to 0.12 and resolved newly introduced lint errors across four files. Changes were cherry-picked from canonical/starbase#409.
85%
#230 chore(deps): update dependency lint/ruff to v0.1.15 craft-cli merged Merged automated dependency update upgrading lint/ruff from v0.1.14 to v0.1.15. Approved by two reviewers, passed CI checks, and maintained 94.81% test coverage across a single file change.
85%