-
Notifications
You must be signed in to change notification settings - Fork 127
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
fix: switch to ruff-format #696
Conversation
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
We have a mix of single and double triple quoted strings in the codebase. This just normalizes on double quoted triple quote strings, following PEP 257's docstring recommendation and following PEP 8's recommendation that all triple quoted strings be made with double quotes (to match PEP 257's docstrings). Before these weren't normalized and could go either way. |
You can also remove black's config from pyproject.toml. |
Signed-off-by: Henry Schreiner <henryschreineriii@gmail.com>
Ahh, thanks! I could have sworn I did that, but I think I just started to and didn't finish. |
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
This is 30x faster than black, drops a dependency, and supports single quotes as a setting.
See https://astral.sh/blog/the-ruff-formatter & scientific-python/cookie#299.