← Back to issue list

style: autoformat to fix for ruff 0.4

View original Github issue

Metadata

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

Current evaluation

Merged after two approvals. Autoformats the codebase to comply with ruff 0.4 via tox -f format, updating 10 files with 27 additions and 46 deletions.

Suggested action:

No scores available.

Issue body

This was created entirely by running `tox -f format` to run all the autoformatters.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after two approvals. Autoformats the codebase to comply with ruff 0.4 via tox -f format, updating 10 files with 27 additions and 46 deletions.
qwen3.6-35b-a3b-mtp-q6 Merged automated formatting changes generated by tox -f format to align with ruff 0.4 requirements. The maintainer applied all autoformatters to the repository, and the update was accepted without comments.
qwen3.6-35b-a3b-mtp-q6 Merged an autoformatting update to align the codebase with ruff 0.4. Changes were generated automatically via tox -f format and applied without manual edits.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#238 style: autoformat with ruff 0.6 starbase merged Merged following approval and passing CI. Updates code formatting using ruff 0.6, affecting a single file. Approved by one reviewer with no unresolved comments.
87%
#255 style: autoformat with ruff 0.9 craft-store merged Merged following two approvals. Applied ruff 0.9 autoformatting across five files, altering 22 lines. All CI checks passed. A maintainer noted that switching from yamllint to prettier will resolve remaining linting concerns.
84%
#4416 style: fix ruff format tox environment snapcraft merged Merged into main after passing CI and two approvals. The PR fixes the ruff format tox environment configuration, changing one file (+2/-3 lines).
81%
#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.
78%
#2074 style: update to ruff 0.9.0 formatting charmcraft merged Merged to align code formatting with ruff 0.9.0 linting rules. Approved by two reviewers and passed CI. One change was noted for upstreaming to starbase instead of applying it locally.
78%
#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.
77%
#2229 style: fix linting errors charmcraft merged Merged changes resolving linting errors from the latest ruff and pyright. Approved by two reviewers with zero unresolved comments. Updated 8 files, removing 22 lines overall.
76%
#119 style: replace black with ruff formatter craft-archives merged Merged after approval by two reviewers. Replaced the Black code formatter with Ruff across 14 files, resulting in 42 additions and 21 deletions. The update successfully modernizes the project's formatting toolchain.
76%
#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.
76%
#580 style(ruff): fix linting issues from ruff update craft-application merged Merged after approval by three reviewers. Resolves linting errors from a ruff update, modifying three files with a +7/-7 line change. Successfully integrated into the codebase.
76%