← Back to issue list

chore: update ruff configuration for 0.8

View original Github issue

Metadata

Project
craft-cli
Number
#317
Type
pull request
State
merged
Author
bepri
Labels
Created
Updated
Closed

Current evaluation

Merged. Removed two unsupported linter rules from pyproject.toml to align with ruff 0.8. Approved by two reviewers, passed CI, and merged after auto-format resolved remaining linting errors.

Suggested action:

No scores available.

Issue body

Two of the referenced linter rules in pyproject were unsupported by the schema. This PR removes them. - [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
qwen/qwen3.6-35b-a3b Merged. Removed two unsupported linter rules from pyproject.toml to align with ruff 0.8. Approved by two reviewers, passed CI, and merged after auto-format resolved remaining linting errors.
qwen3.6-35b-a3b-mtp-q6 Merged after removing two unsupported ruff linter rules from pyproject to align with version 0.8. Auto-formatting resolved final linting errors prior to merge.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
85%
#700 chore: update ruff craft-parts merged 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.
85%
#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.
84%
#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.
84%
#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.
84%
#589 chore(deps): update dependency ruff to v0.1.14 craft-parts merged Merged automated dependency update for ruff from v0.1.6 to v0.1.14. Approved by two reviewers with passing CI checks. Updated a single configuration file to apply the new version constraint.
83%
#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.
83%
#214 chore(deps): update dependency lint/ruff to v0.1.7 craft-cli merged Merged an automated dependency update upgrading lint/ruff from v0.1.6 to v0.1.7. Approved by two reviewers, passed CI checks, maintained 94.77% test coverage, and modified a single file before final merge.
83%
#423 chore(deps): update dependency ruff to v0.1.6 rockcraft merged Merged automated dependency update upgrading ruff from v0.1.1 to v0.1.6. Approved by two reviewers, passed CI checks, and applied with a minimal two-line change across two files.
83%
#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.
83%