If you're using this, try
https://github.com/alexcouper/captainhook
At the time of writing it provides git hook checks for pdb, pep8 and python3 syntax straight out of the box.
Last active
December 23, 2020 07:18
-
-
Save alexcouper/9421502 to your computer and use it in GitHub Desktop.
My pre commit hook. No more pdb please!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| # This hook was inspired by the work done on these two blogs: | |
| # - http://blog.keul.it/2013/11/no-more-pdbsettrace-committed-git-pre.html | |
| # - http://codeinthehole.com/writing/tips-for-using-a-git-pre-commit-hook/ | |
| # Ensure that code that isn't part of the prospective commit isn't tested | |
| # within the pre-commit script using git stash. | |
| git stash -q --keep-index | |
| # Test prospective commit | |
| FILES_PATTERN='\.py(\..+)?$' | |
| FORBIDDEN_PATTERN='^[^#]*pdb.set_trace()' | |
| FORBIDDEN_FILES=`git diff --cached --name-only | \ | |
| grep -E $FILES_PATTERN | \ | |
| GREP_COLOR='4;5;37;41' xargs grep --color --with-filename -n \ | |
| -e $FORBIDDEN_PATTERN` | |
| [ -n "$FORBIDDEN_FILES" ] && echo "COMMIT REJECTED \n$FORBIDDEN_FILES" | |
| RETVAL=$? | |
| git stash pop -q | |
| [ $RETVAL -eq 1 ] && exit 0 |
I would have thought you would have used the grep command --colour instead of --color 🇬🇧 😏
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Nice!