← Back to issue list

style: replace black with ruff formatter

View original Github issue

Metadata

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

Current evaluation

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.

Suggested action:

No scores available.

Issue body

- [ ] Have you followed the guidelines for contributing? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `tox`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b 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.
qwen3.6-35b-a3b-mtp-q6 Merged a pull request replacing the Black code formatter with Ruff. The change was accepted and integrated into the main branch without additional comments or revisions.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
92%
#1084 build(style): switch from black to ruff craft-parts merged 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.
86%
#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.
79%
#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.
77%
#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.
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%
#1662 style: autoformat to fix for ruff 0.4 charmcraft merged 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.
76%
#305 Remove black craft-cli merged Removed the black dependency due to Python 3.8 incompatibility, replacing it with ruff for code formatting. The change was transposed from charmcraft, approved by two reviewers, and merged after passing CI checks.
75%
#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%
#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.
75%