Last active
August 29, 2015 14:15
-
-
Save pepf/ffb84ee8c4aa0a26f3f5 to your computer and use it in GitHub Desktop.
precommit hook: search commit for debug statements
This file contains 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
#!/bin/sh | |
# Provide the canonicalize filename (physical filename with out any symlinks) | |
# like the GNU version readlink with the -f option regardless of the version of | |
# readlink (GNU or BSD). | |
# This file is part of a set of unofficial pre-commit hooks available | |
# at github. | |
# Link: https://github.com/githubbrowser/Pre-commit-hooks | |
# Contact: David Martin, david.martin.mailbox@googlemail.com | |
########################################################### | |
# There should be no need to change anything below this line. | |
# Canonicalize by recursively following every symlink in every component of the | |
# specified filename. This should reproduce the results of the GNU version of | |
# readlink with the -f option. | |
# | |
# Reference: http://stackoverflow.com/questions/1055671/how-can-i-get-the-behavior-of-gnus-readlink-f-on-a-mac | |
canonicalize_filename () { | |
local target_file="$1" | |
local physical_directory="" | |
local result="" | |
# Need to restore the working directory after work. | |
local working_dir="`pwd`" | |
cd -- "$(dirname -- "$target_file")" | |
target_file="$(basename -- "$target_file")" | |
# Iterate down a (possible) chain of symlinks | |
while [ -L "$target_file" ] | |
do | |
target_file="$(readlink -- "$target_file")" | |
cd -- "$(dirname -- "$target_file")" | |
target_file="$(basename -- "$target_file")" | |
done | |
# Compute the canonicalized name by finding the physical path | |
# for the directory we're in and appending the target file. | |
physical_directory="`pwd -P`" | |
result="$physical_directory/$target_file" | |
# restore the working directory after work. | |
cd -- "$working_dir" | |
echo "$result" | |
} |
This file contains 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
#!/bin/sh | |
# Git pre-commit hook that runs multiple hooks specified in $HOOKS. | |
# Make sure this script is executable. Bypass hooks with git commit --no-verify. | |
# This file is part of a set of unofficial pre-commit hooks available | |
# at github. | |
# Link: https://github.com/githubbrowser/Pre-commit-hooks | |
# Contact: David Martin, david.martin.mailbox@googlemail.com | |
########################################################### | |
# SETTINGS: | |
# pre-commit hooks to be executed. They should be in the same .git/hooks/ folder | |
# as this script. Hooks should return 0 if successful and nonzero to cancel the | |
# commit. They are executed in the order in which they are listed. | |
#HOOKS="pre-commit-default pre-commit-compile pre-commit-uncrustify" | |
HOOKS="pre-commit-default pre-commit-blacklist" | |
########################################################### | |
# There should be no need to change anything below this line. | |
. "$(dirname -- "$0")/canonicalize_filename.sh" | |
# exit on error | |
set -e | |
# Absolute path to this script, e.g. /home/user/bin/foo.sh | |
SCRIPT="$(canonicalize_filename "$0")" | |
# Absolute path this script is in, thus /home/user/bin | |
SCRIPTPATH="$(dirname -- "$SCRIPT")" | |
for hook in $HOOKS | |
do | |
#echo "Running hook: $hook" | |
# run hook if it exists | |
# if it returns with nonzero exit with 1 and thus abort the commit | |
if [ -f "$SCRIPTPATH/$hook" ]; then | |
"$SCRIPTPATH/$hook" | |
if [ $? != 0 ]; then | |
exit 1 | |
fi | |
else | |
echo "Error: file $hook not found." | |
echo "Aborting commit. Make sure the hook is in $SCRIPTPATH and executable." | |
echo "You can disable it by removing it from the list in $SCRIPT." | |
echo "You can skip all pre-commit hooks with --no-verify (not recommended)." | |
exit 1 | |
fi | |
done |
This file contains 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
#!/bin/sh | |
if precommit-diffblacklist.py; then | |
exit 0 | |
else | |
exit 1 | |
fi |
This file contains 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
#!/usr/bin/env python | |
import os | |
import re | |
import sys | |
#blacklist | |
blacklist = [ | |
"console.log", | |
"var_dump", | |
"print_r", | |
"die" | |
] | |
#Get current git diff | |
p = os.popen("git diff HEAD","r") | |
filename = "" | |
while 1: | |
line = p.readline() | |
if not line: break | |
if line.startswith("+++"): | |
filename = line.replace("+++ b/", "") | |
if line.startswith("+"): | |
#check for rules that have been added, containing words from blacklist: | |
if any(x in line for x in blacklist): | |
print "debug statement found in file: %s" % filename | |
print line | |
sys.exit(1) | |
#no errors yeah | |
sys.exit(0) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment