Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(dev): replace black with ruff #525

Merged
merged 1 commit into from
Jan 30, 2024
Merged

Conversation

JP-Ellis
Copy link
Contributor

@JP-Ellis JP-Ellis commented Jan 30, 2024

πŸ“ Summary

Ruff has included (for a few months now) a formatter functionality. The formatting and linting are entirely distinct functionalities, though both provided by Ruff. My experience so far has been that Ruff's formatting is very fast, and mostly matches Black's formatting.

🚨 Breaking Changes

πŸ”₯ Motivation

Ruff is super fast ⚑

While Black is already very fast, using a single tool that is fast and consistent seems like a better option going forward.

πŸ”¨ Test Plan

Regular CI/CD

πŸ”— Related issues/PRs

None

@JP-Ellis JP-Ellis force-pushed the chore/replace-black-with-ruff branch from 17d51cd to 0e2e5fd Compare January 30, 2024 00:18
Ruff has included (for a few months now) a formatter functionality. The
formatting and linting are entirely distinct functionalities, though
both provided by Ruff. My experience so far has been that Ruff's
formatting is very fast, and mostly matches Black's formatting.

Signed-off-by: JP-Ellis <josh@jpellis.me>
@JP-Ellis JP-Ellis force-pushed the chore/replace-black-with-ruff branch from 0e2e5fd to 6f9fa0d Compare January 30, 2024 00:24
@JP-Ellis JP-Ellis merged commit dc73777 into master Jan 30, 2024
24 of 36 checks passed
@JP-Ellis JP-Ellis deleted the chore/replace-black-with-ruff branch January 30, 2024 00: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.

None yet

1 participant