← Back to issue list

chore(linter): ruff auto fix base on starbase config

View original Github issue

Metadata

Project
craft-parts
Number
#581
Type
pull request
State
merged
Author
syu-w
Labels
Created
Updated
Closed

Current evaluation

Merged automated ruff fixes applying the starbase config across 156 files. Adjusts formatting and import order, with tweaks for trailing commas and ignored rules. Approved by three reviewers and depends on PR #580.

Suggested action:

No scores available.

Issue body

All auto fix. Depends on https://github.com/canonical/craft-parts/pull/580

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged automated ruff fixes applying the starbase config across 156 files. Adjusts formatting and import order, with tweaks for trailing commas and ignored rules. Approved by three reviewers and depends on PR #580.
qwen3.6-35b-a3b-mtp-q6 Merged after applying Ruff auto-fixes based on the starbase configuration. Trailing comma formatting was corrected, and import order changes from previously ignored rules were noted for follow-up PRs.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#580 chore(linter): enable ruff base on starbase config craft-parts merged Merged a chore enabling the ruff linter based on the starbase configuration. Approved by two reviewers, the update modified six files with 224 additions and 134 deletions.
87%
#588 chore: use the latest ruff config from starbase craft-application merged Merged an update to the ruff configuration using the latest starbase settings. Approved by two reviewers across 86 files, supporting the craft-application migration to starflow.
84%
#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.
83%
#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.
81%
#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.
80%
#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.
80%
#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.
80%
#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.
80%
#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.
78%
#651 chore(linter): refine ruff configurations craft-parts merged Merged into main after passing CI and receiving two approvals. The change restores previously removed Pylint rules to the Ruff linter configuration, with one rule temporarily commented out pending upstream updates.
78%