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

add linter/formatter to enforce coding style #72

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mishaschwartz
Copy link
Contributor

  • add ruff as a dev dependency to format and lint files
  • add pre-commit as a dev dependency to run ruff on commit and a workflow to run it on github as well

Note

This PR does not actually format/lint any of the existing code base yet!
I would prefer to wait until #64 is merged to master (since it is such a large PR) and then we can format everything properly in a commit on this branch and then merge this PR

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