-
Notifications
You must be signed in to change notification settings - Fork 20
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
Create make precommit
command
#220
Comments
started working on this! |
@danaelhe I think we don't have to make a command for this. pre-commit are hooks that invoke specified operations before committing automatically. |
one way is to map the pre-commit to a script and run these and other commands you like before the commit. |
If anyone feels like raising a PR, they can do it. From my side, it can take some time to complete. |
@danaelhe can you please explain more about the |
Oh that's a way better idea! Go for it 👍 Thanks for the suggestion. |
Working with the first approach! |
resolved in #228 |
Issue
The CI/CD pipeline includes a linter test that must pass. It would be much easier if we baked
poetry run black tests
into amake precommit
command in order for theblack
package to reformat the files before passing them through the CI/CD linter.The text was updated successfully, but these errors were encountered: