← 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 Ruff autofixes across 156 files to align craft-parts with Starbase's lint configuration; change landed as commit 30b73d97.

Resolution: closed resolved

Reason: PR #581 was approved and merged as commit 30b73d97, applying Ruff autofixes after enabling the Starbase-based configuration in #580.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged Ruff autofixes across 156 files to align craft-parts with Starbase's lint configuration; change landed as commit 30b73d97.
stealth/space-bunny-alpha — closed resolved Merged PR #581 as commit 30b73d97, applying Ruff autofixes across 156 files after Starbase-based Ruff configuration landed in #580; COM812 was omitted.
stealth/space-bunny-alpha — closed resolved Merged Ruff autofixes aligned with Starbase configuration, including removal of COM812; 156 files changed (+244/-389).
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 work

  • Related To: canonical/craft-parts#580 (confidence 99%)

    Prerequisite PR that enabled Ruff based on Starbase configuration, merged as commit 111c8907.

Related issues

Issue Project State Summary Similarity
#103 chore: fix new lints from ruff 0.12 craft-grammar merged Merged PR #103 as a3ef1dd, updating Ruff to 0.12 and fixing resulting lint errors across four files; cherry-picked from canonical/starbase#409.
82%
#580 chore(linter): enable ruff base on starbase config craft-parts merged Merged in #580 via commit 111c8907, enabling Ruff based on the Starbase configuration.
82%
#588 chore: use the latest ruff config from starbase craft-application merged Merged PR #588 as commit 44b1a940, adopting starbase's latest Ruff configuration and updating linting across 86 files for the Starflow migration.
81%
#574 chore(linter): rebase on starbase linters config craft-parts closed Closed unmerged for redo; the narrower Starbase Ruff migration landed in #580 (111c8907), with autofixes in #581 (30b73d97).
80%
#1662 style(lint): auto-fix with ruff craft-parts merged Merged Ruff auto-fixes across six files after maintainer review; canonical/craft-parts#1662 landed as commit 2be2b462 (+80/-58).
79%
#22 chore(lint): update ruff and config starbase merged Merged in canonical/starbase#22 via commit f3d35a6, updating Ruff and its config to ignore excessive test-function arguments (PLR0913).
78%
#36 chore: switch to ruff craft-platforms merged Merged PR #36 as commit ce5f6b9, switching linting and formatting to Ruff, updating configuration, and applying fixes across seven files.
77%
#568 chore(linter): rebase on starbase linters config craft-parts closed Closed unmerged after review requests; the Starbase Ruff migration was subsequently implemented more narrowly by PR #580 in commit 111c8907.
76%
#1347 chore(deps): update ruff and fix new linting errors charmcraft merged Ruff pre-commit hook was bumped and resulting lint errors auto-fixed across 10 files (+40/-37). PR #1347 was merged by lengau on 2023-10-23 as commit 34018724.
76%
#700 chore: update ruff craft-parts merged Merged PR #700 updated Ruff configuration across four files, superseding #699; a follow-up lint issue was fixed in merged PR #702.
76%