← Back to issue list

style(lint): fix ruff 0.6.0 linting errors

View original Github issue

Metadata

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

Current evaluation

Merged PR #1825 as commit 7aa2965b, applying Ruff 0.6.0 autofixes across 31 files to resolve linting errors.

Resolution: closed resolved

Reason: PR #1825 was merged after maintainer approval as commit 7aa2965b. The commit applies Ruff's autofixes across 31 files, resolving the Ruff 0.6.0 lint failures.

Evaluated by: mr-cal-bot

No scores available.

Issue body

This entirely uses ruff's autofix

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #1825 as commit 7aa2965b, applying Ruff 0.6.0 autofixes across 31 files to resolve linting errors.
stealth/space-bunny-alpha — closed resolved Merged PR #1825 as commit 7aa2965b, applying Ruff 0.6.0 autofixes across 31 files (+70/-71) to resolve linting errors.
qwen/qwen3.6-35b-a3b — — Merged following two reviewer approvals and passing CI. The change resolves ruff 0.6.0 linting errors across 31 files using ruff's autofix, resulting in 70 additions and 71 deletions.
qwen3.6-35b-a3b-mtp-q6 — — Merged changes resolving ruff 0.6.0 linting errors by applying ruff's autofix.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#423 style(lint): fix ruff 0.6.0 linting errors craft-application merged Merged PR #423 as commit 2f56f14b, applying ruff check --fix across 21 files (+70/-70) to clear the new Ruff 0.6.0 lint errors. Approved by 2 maintainer reviewers.
94%
#4981 style(lint): fix ruff 0.6 linting errors snapcraft merged Merged PR #4981 as commit 3a1b8a949, fixing Ruff 0.6 lint errors with a minimal one-file change.
94%
#29 style(lint): fix ruff 0.6.0 linting errors craft-platforms merged Merged PR #29 as commit 399c4e0, fixing Ruff 0.6.0 lint errors and switching the pre-commit Ruff hook to rolling versions.
92%
#815 fix linting issues for ruff 0.6 craft-parts merged Merged PR #815, applying Ruff 0.6 autofixes and manual lint corrections across 44 files (+98/-98).
92%
#624 style(lint): fix ruff 0.6.0 linting errors craft-providers merged Merged as commit 823622d, fixing Ruff 0.6.0 lint errors and switching the pre-commit Ruff hook to rolling versions.
91%
#1737 style: fix linting for ruff 0.5 charmcraft merged Merged PR #1737 as commit 8994523e, updating one file (+2/-6) to fix linting under Ruff 0.5.
90%
#173 style: fix lint for ruff 0.12 craft-archives merged Merged PR #173 as commit 53d732e, fixing lint for Ruff 0.12 while retaining Ruff 0.11 compatibility; all CI checks passed.
87%
#357 style: fix lints from ruff 0.12 craft-cli merged Merged as dd635c9, fixing new Ruff 0.12 lint errors across seven files (+11/-10); all CI checks passed.
87%
#205 style(lint): autofix ruff 0.6.4 linting errors craft-store closed PR #205 was closed unmerged after 745 days; its Ruff 0.6.4 ARG001 autofix received no review, comments, or CI runs, and no resolving commit was recorded.
87%
#1156 style: fix linting errors for ruff 0.12.0 craft-parts merged Ruff 0.12.0 lint fixes merged as PR #1156 (commit eb82b792), correcting style violations in 5 files (+9/-12) after the ruff-pre-commit v0.12.0 bump in #407; all CI checks passed.
87%