-
Notifications
You must be signed in to change notification settings - Fork 256
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: restructure linting & formatting #252
Conversation
The |
Based on the current runs this has been a 1/3 reduction in time needed for the linting&formatting workflow (~55s down to ~35s) + elimination of every workflow trigger based on a separate "I forgot linting & formatting" commit. 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
black
andpylint
toruff
in poetry environmentpre-commit
to local dev workflow for automatic linting&formatting before a commitI've run all tests and triple-checked the entire repo but please check again if possible for any braking changes.