← Back to issue list

style(lint): auto-fix with ruff

View original Github issue

Metadata

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

Current evaluation

Merged Ruff auto-fixes across six files after maintainer review; canonical/craft-parts#1662 landed as commit 2be2b462 (+80/-58).

Resolution: closed resolved

Reason: PR #1662 was approved by two reviewers and merged as commit 2be2b462, with all listed CI checks passing.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Some of these were unsafe fixes so we need to review carefully. - [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 `make lint && make test`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? ---

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged Ruff auto-fixes across six files after maintainer review; canonical/craft-parts#1662 landed as commit 2be2b462 (+80/-58).
stealth/space-bunny-alpha — closed resolved Merged canonical/craft-parts#1662 as commit 2be2b462, applying Ruff’s auto-fixes across six files (+80/−58) after maintainer review.
qwen/qwen3.6-35b-a3b — — Merged after two approvals and passing all CI checks. Automatically applies ruff linting fixes across six files, carefully reviewing unsafe changes to update code style without altering functionality.
qwen/qwen3.6-35b-a3b
Staleness: 0
Complexity: 0
Confidence: 95
needs review Applies automated ruff linting fixes across 6 files. All CI checks pass, and the PR has received two approvals with no unresolved comments. Ready for maintainer merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 10
Confidence: 95
needs review Auto-fixes linting and style issues using ruff across 6 files. Approved by two reviewers with one unresolved comment. All CI checks pass.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 5
Confidence: 90
needs review Auto-fixes linting issues using ruff, including unsafe fixes. PR has two approvals, one unresolved comment, and all CI checks passing. Awaiting final maintainer review to address comments and merge.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 5
Confidence: 90
needs review Applies ruff auto-fixes across 6 files to update project linting. Currently approved by one reviewer with one unresolved comment, CI mostly passing, and awaiting final maintainer review.
qwen/qwen3.6-35b-a3b
Staleness: 5
Complexity: 15
Confidence: 85
needs review Applies ruff auto-fixes to style and lint the codebase. Includes some unsafe fixes requiring careful review. Currently pending maintainer review and CI completion.

Update history

Date Change
updated
updated
updated
updated
created

Related issues

Issue Project State Summary Similarity
#1825 style(lint): fix ruff 0.6.0 linting errors charmcraft merged Merged PR #1825 as commit 7aa2965b, applying Ruff 0.6.0 autofixes across 31 files to resolve linting errors.
84%
#580 style(ruff): fix linting issues from ruff update craft-application merged Merged PR canonical/craft-application#580 (commit 97347705) by lengau applied style-only fixes (+7/-7, 3 files) for new lint failures caused by a Ruff update; approved by 3 reviewers.
84%
#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.
83%
#901 style: fix ruff 0.13 linting issues craft-application merged Merged in canonical/craft-application#901 via commit 0a659742, fixing Ruff 0.13 linting with a one-line change; review was approved and all CI checks passed.
83%
#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.
83%
#1465 style: fix linting issues for newer ruff craft-parts merged Merged #1465 as a58695ef, fixing five files (+13/-6) for compatibility with newer Ruff; all CI checks passed.
82%
#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.
82%
#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).
81%
#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.
81%
#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.
81%