← Back to issue list

build(style): switch from black to ruff

View original Github issue

Metadata

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

Current evaluation

Merged after approval by two reviewers and passing all CI checks. Switches the code formatter from Black to Ruff via two commits that update configuration and autoformat 32 files.

Suggested action:

No scores available.

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? ----- This has two commits: the first does the switch and the second autoformats

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after approval by two reviewers and passing all CI checks. Switches the code formatter from Black to Ruff via two commits that update configuration and autoformat 32 files.
qwen3.6-35b-a3b-mtp-q6 Merged maintainer lengau's changes switching the codebase formatter from Black to Ruff. Two commits implement the tool switch and apply automatic formatting across the project.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
86%
#212 style: replace black formatter with ruff format starbase merged Merged the replacement of Black with Ruff format. Approved by two reviewers, the change updates seven files with 26 additions and 10 deletions to standardize project styling.
84%
#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.
79%
#1960 style(lint): move from black to ruff charmcraft merged Merged after two approvals and passing CI. Replaced Black with Ruff across 122 files, updating 1778 lines and removing 557 to modernize the linting configuration.
78%
#363 chore: remove use of black in favor of ruff starbase merged Merged a maintainer's change removing dangling references to the black formatter and replacing it with ruff. Approved by two reviewers, passed CI, and modified one file with minimal changes.
77%
#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.
77%
#236 build: update ruff settings craft-cli merged Merged following approval from two reviewers and passing CI checks. The change updates ruff configuration settings with a five-line modification to align linting rules.
76%
#5343 style: ruff 0.11 fixes snapcraft merged Merged minor style updates to align with ruff 0.10 and 0.11. Approved by two reviewers, passed CI checks, and merged with minimal changes across one file.
76%
#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.
75%
#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.
75%