← Back to issue list

style: migrate from black to ruff for formatting

View original Github issue

Metadata

Project
craft-providers
Number
#439
Type
pull request
State
closed
Author
mr-cal
Labels
—
Created
Updated
Closed

Current evaluation

Ruff formatting migration was abandoned unmerged after the team considered `ruff format` too experimental.

Resolution: closed stale

Reason: On 2023-11-21, maintainer mr-cal closed the PR unmerged, citing a team decision that `ruff format` was too experimental. No resolving commit or superseding PR was identified.

Evaluated by: mr-cal-bot

No scores available.

Issue body

- [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 `tox`? ----- Move from black to ruff for formatting (more details [here](https://astral.sh/blog/the-ruff-formatter)). It looks like ruff has done a great job of making a black-compatible formatter. The only difference in this repo is a warning for strings that exceed the line length.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed stale Ruff formatting migration was abandoned unmerged after the team considered `ruff format` too experimental.
qwen/qwen3.6-35b-a3b — closed stale PR closed without merging after maintainer noted team decision that ruff format was too experimental.

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#146 migrate to `ruff` craft-cli closed Closed without a documented fix; Ruff was later enabled via #204 (commit 09b91ba), with Black replaced by Ruff formatting in #212.
82%
#119 style: replace black with ruff formatter craft-archives merged Merged PR #119 replaced Black with the Ruff formatter across 14 files (+42/-21), completing the style migration.
78%
#212 style: replace black formatter with ruff format starbase merged Merged PR #212 as commit aae9d00, replacing Black with Ruff format across seven files (+26/-10).
77%
#739 style: switch from black to ruff craft-providers merged Merged #739 as commit 2b9c894, replacing Black with Ruff formatting across 23 files in preparation for Starbase adoption.
74%
#1084 build(style): switch from black to ruff craft-parts merged Merged #1084 as commit b28a1dfc, replacing Black with Ruff and autoformatting 32 files; all CI checks passed.
73%
#238 style: autoformat with ruff 0.6 starbase merged Merged PR #238 as commit 6bfb969, applying Ruff 0.6 autoformatting to one file.
71%