-
Notifications
You must be signed in to change notification settings - Fork 9
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
Set up basic pre-commits #77
Conversation
* black (source code formatting) * syntax validation * basic flake8 checks Add an entry point to enable pre-commits if the package is installed in a dials environment.
3 similar comments
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.
pip install pre-commit
and pre-commit install
seem to do something, and no errors! So I'll assume this works. We should probably have documentation notes somewhere for developers about these steps.
Good point. I've added some documentation blurb. If you're happy with this then it can go in. |
Yes, that looks good! Thank you! |
Add an entry point to enable pre-commits if the package is installed
in a dials environment.
You'll have to see how that works in your development environment.
It should be as simple as
pip install pre-commit
followed bypre-commit install
in the repository directory.