← 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 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.

Suggested action: —

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
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
#423 style(lint): fix ruff 0.6.0 linting errors craft-application merged Merged after two approvals and passing CI. Resolves ruff 0.6.0 linting errors across 21 files using ruff check --fix, applying automated style corrections to align with the updated linter version.
91%
#1825 style(lint): fix ruff 0.6.0 linting errors charmcraft merged 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.
88%
#4981 style(lint): fix ruff 0.6 linting errors snapcraft merged Merged after receiving two approvals and passing CI checks. Resolves ruff 0.6 linting errors with a minimal one-file change.
87%
#1577 style(lint): enable some previously-ignored ruff rules charmcraft merged Merged maintainer lengau's changes enabling previously ignored ruff linting rules. Approved by two reviewers, the update modifies ten files (+31/-45 lines) and addresses seven review comments, enforcing stricter code style.
86%
#357 style: fix lints from ruff 0.12 craft-cli merged Merged following approval from three reviewers and successful CI checks. Resolved lint errors introduced by Ruff 0.12 across seven files with minor code adjustments.
86%
#1737 style: fix linting for ruff 0.5 charmcraft merged Merged after approval by two reviewers and passing CI. The change updates the codebase to comply with ruff 0.5 linting rules, adjusting style configurations and fixing formatting across one file.
85%
#1156 style: fix linting errors for ruff 0.12.0 craft-parts merged Merged after receiving approval from two reviewers and passing all CI checks. The update resolves linting errors introduced by ruff 0.12.0 across five files, adjusting code formatting to comply with the new linter version.
84%
#1465 style: fix linting issues for newer ruff craft-parts merged Merged after two approvals and passing all CI checks. Resolves linting compatibility issues with newer ruff versions, updating five files.
84%
#969 style: fix linting issues for ruff 0.9 craft-parts merged Merged a minor style fix addressing linting issues for ruff 0.9. Approved by two reviewers, passed CI checks, and updated two files with a net change of one line.
82%
#803 style: fix linting issues with ruff 0.12 craft-application merged Merged after approval by two reviewers. Resolves linting failures introduced by ruff 0.12 across 14 files. CI passed for fast tests, linting, and security scans, while slow test failures were tracked separately in #804.
82%