-
Notifications
You must be signed in to change notification settings - Fork 104
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
Formatting: black
-> ruff format
#506
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #506 +/- ##
=======================================
Coverage 88.84% 88.84%
=======================================
Files 35 35
Lines 3514 3514
Branches 484 484
=======================================
Hits 3122 3122
Misses 284 284
Partials 108 108 ☔ View full report in Codecov by Sentry. |
1112002
to
cbdb89f
Compare
Any rationale for replacing black with ruff? |
@piotr-dobrogost While we're here: Anything against |
Thanks for the answer. |
@piotr-dobrogost Good idea. I will note the reasoning in the changelog (updated the PR as well) |
6d1e0a0
to
15efb6a
Compare
Changes
Development
Convert from
black
toruff format
(https://docs.astral.sh/ruff/formatter/)This retains the formatting style of
black
while eliminating a dev dependency by using our existing rust-basedruff
linter.