-
Notifications
You must be signed in to change notification settings - Fork 24
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
Add pre-commit with ruff + black #30
Comments
@Ovsyanka83, I would like to work on this issue, just give me an overview on what needs to be done. |
We need to add a pre-commit file that will run ruff and black. Note that both must use configurations from project.toml. Let's leave out pyright for now. We'll just add it as a separate step in CI later because there are many issues with running it at pre-commit. |
I'm afraid @coffeewasmyidea did this much faster. I hope it's okay with you. |
@Ovsyanka83, It's fine, let me know if you need help in another issue. |
@Aditi840 Oh I always do, thanks! There's also the linked issue that is closely related to this one :) So there are plenty of coding, documentation, testing, and infrastructural tasks. If you want something more researchy, we also have the two milestones that require research, planning, and a lot of coding as a result. |
No description provided.
The text was updated successfully, but these errors were encountered: