-
Notifications
You must be signed in to change notification settings - Fork 28
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
Pre-commit addition is heavy #75
Comments
🤔 didnt realise go was needed for one of them. I guess that would be
|
Can we ignore gitleaks? It does not feel relevant to the project. We are not likely to have password file right? |
I think its safe-ish to remove. The idea is to prevent tokens and secrets from mistakenly being committed. But I think that is more of a need on a larger team rather than a project maintained by an individual |
See #77. |
I tried to update to the new main this morning.
poetry run pre-commit install --install-hooks
fails for meWhy do I need
go
? What is that giving the project that is helpful?The text was updated successfully, but these errors were encountered: