Skip to content

🔨 replace Black with Ruff formatter#81

Merged
Miyamura80 merged 4 commits into
mainfrom
chore/ruff-format
Jan 26, 2026
Merged

🔨 replace Black with Ruff formatter#81
Miyamura80 merged 4 commits into
mainfrom
chore/ruff-format

Conversation

@Miyamura80

Copy link
Copy Markdown
Owner

Summary

  • replace Black formatter usage with ruff format in tooling and docs
  • remove Black dependency and refresh uv.lock
  • fix JSON formatting prune expression in Makefile

Testing

  • make fmt

@greptile-apps

greptile-apps Bot commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Greptile Overview

Greptile Summary

This PR successfully migrates the codebase from Black to Ruff formatter. The changes are comprehensive and well-executed:

  • Removed Black dependency from pyproject.toml and uv.lock
  • Updated Makefile to use ruff format instead of black, including removal of Black from install_tools target
  • Fixed IGNORE_LINT_DIRS syntax from pipe-separated to space-separated format
  • Corrected FIND_PRUNE expression for proper JSON file filtering in make fmt
  • Updated all documentation (README.md, CLAUDE.md, manual_docs/branch_comparison.md, .cursor/rules/makefile-commands.md) to reflect the change
  • Applied Ruff formatting to test files, showing different formatting preferences (multi-line assert messages)
  • Applied formatting to docs JSON/TypeScript files

The migration is clean with no logical issues. Ruff's existing configuration in pyproject.toml (line-length = 88) matches Black's previous settings, ensuring formatting consistency.

Confidence Score: 5/5

  • This PR is safe to merge with no issues
  • The migration from Black to Ruff formatter is complete and thorough. All references to Black have been properly removed, documentation has been updated consistently across the codebase, and the Makefile changes are correct. The formatting changes to test files are purely cosmetic (Ruff's style for multi-line strings differs slightly from Black's). No logical bugs or breaking changes were introduced.
  • No files require special attention

Important Files Changed

Filename Overview
Makefile replaced Black with Ruff formatter, fixed IGNORE_LINT_DIRS syntax and FIND_PRUNE expression
README.md updated documentation and feature table to replace Black with Ruff
manual_docs/branch_comparison.md updated feature table to replace Black formatter with Ruff formatter
pyproject.toml removed Black dependency from project dependencies
uv.lock removed Black dependency from lock file

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment thread README.md
|---------|:------:|:------:|
| UV + Pydantic config | ✅ | ✅ |
| CI/Linters (Ruff, Black, Vulture) | ✅ | ✅ |
| CI/Linters (Ruff, Vulture) | ✅ | ✅ |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

check that manual_docs/branch_comparison.md:28 is updated to reflect Ruff instead of Black formatter for consistency

| Black formatter | ✅ | ✅ | Opinionated code formatting |
Prompt To Fix With AI
This is a comment left during a code review.
Path: README.md
Line: 44:44

Comment:
check that `manual_docs/branch_comparison.md:28` is updated to reflect Ruff instead of Black formatter for consistency

```
| Black formatter | ✅ | ✅ | Opinionated code formatting |
```

How can I resolve this? If you propose a fix, please make it concise.

@Miyamura80 Miyamura80 merged commit 15f5192 into main Jan 26, 2026
10 checks passed
@github-actions github-actions Bot deleted the chore/ruff-format branch January 26, 2026 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant