-
Notifications
You must be signed in to change notification settings - Fork 87
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
Added pre-commit for running linters #67
Comments
This comment has been minimized.
This comment has been minimized.
What do you mean? Run it in the CI directly |
CI as a solution. Thanks :) I will add ci file for gitlab. |
I think that i have better idea. Right now all linters and static checker work with docker. It's works like that because i wonted that user after run |
Add into .git/hooks/pre-commit:
After
For example?) |
there's a tool which makes it more generic: https://pre-commit.com/ |
This is good feature but we running all linters from docker and it's not fast. Does anyone have any thoughts about this?
The text was updated successfully, but these errors were encountered: