UkraineNow-Intel / autoSA-backend

Django backend for autoSA
0 stars 1 forks source link

Added pre-commit hooks and instructions. #22

Closed j-bennet closed 2 years ago

j-bennet commented 2 years ago

This adds a pre-commit hook to prevent commiting things to master or main. Since we're on free github plan, we can't have protected branches. Better safe than sorry.

Pre-commit is a nice tool, we ask people to use it on pgcli to enforce black.

Here, I added only 2 hooks, one to prevent commiting to master or main, and another to prevent pdb statements.

Take a look at this list of supported hooks though, maybe you'd like to add something else:

https://github.com/pre-commit/pre-commit-hooks#hooks-available

cc @ZackPlauche @D0nPiano @robintibor @mgavish, you all commited to this repo, and I think you should be aware of this change.