-
-
Save xbx/1533842 to your computer and use it in GitHub Desktop.
My pre-commit hook for git, check non-ascii-files, jshint, csshint
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
#!/bin/sh | |
# | |
# An example hook script to verify what is about to be committed. | |
# Called by "git commit" with no arguments. The hook should | |
# exit with non-zero status after issuing an appropriate message if | |
# it wants to stop the commit. | |
# | |
# To enable this hook, rename this file to "pre-commit". | |
if git rev-parse --verify HEAD >/dev/null 2>&1 | |
then | |
against=HEAD | |
else | |
# Initial commit: diff against an empty tree object | |
against=4b825dc642cb6eb9a060e54bf8d69288fbee4904 | |
fi | |
# If you want to allow non-ascii filenames set this variable to true. | |
allownonascii=$(git config hooks.allownonascii) | |
# Cross platform projects tend to avoid non-ascii filenames; prevent | |
# them from being added to the repository. We exploit the fact that the | |
# printable range starts at the space character and ends with tilde. | |
if [ "$allownonascii" != "true" ] && | |
# Note that the use of brackets around a tr range is ok here, (it's | |
# even required, for portability to Solaris 10's /usr/bin/tr), since | |
# the square bracket bytes happen to fall in the designated range. | |
test "$(git diff --cached --name-only --diff-filter=A -z $against | | |
LC_ALL=C tr -d '[ -~]\0')" | |
then | |
echo "Error: Attempt to add a non-ascii file name." | |
echo | |
echo "This can cause problems if you want to work" | |
echo "with people on other platforms." | |
echo | |
echo "To be portable it is advisable to rename the file ..." | |
echo | |
echo "If you know what you are doing you can disable this" | |
echo "check using:" | |
echo | |
echo " git config hooks.allownonascii true" | |
echo | |
exit 1 | |
fi | |
## Common | |
EXIT_CODE=0 | |
REPO=$(pwd) | |
# | |
# Checking trailing whitespaces and indent | |
# | |
git diff --cached --check --color=always | |
EXIT_STATUS=$? | |
if [[ $EXIT_STATUS -ne 0 ]]; then | |
EXIT_CODE=$((${EXIT_CODE} + $EXIT_STATUS)) | |
fi | |
# | |
# Begin JsHint hook | |
# A pre-commit hook for git to lint JavaScript files with jshint | |
# @see https://github.com/jshint/jshint/ | |
# | |
JSHINT_BIN=/usr/bin/jshint | |
for FILE in `git diff-index --name-only ${against} -- | egrep *.js`; do | |
if test -f ${REPO}/${FILE}; then | |
${JSHINT_BIN} ${REPO}/${FILE} | |
else | |
continue | |
fi | |
# could similarly wrap Rhino or Node... | |
EXIT_CODE=$((${EXIT_CODE} + $?)) | |
done | |
# End JsHint hook | |
# | |
# Begin CssLint hook | |
# | |
CSSLINT_BIN=/usr/bin/csslint | |
for FILE in `git diff-index --name-only ${against} -- | egrep *.css`; do | |
if test -f ${REPO}/${FILE}; then | |
${CSSLINT_BIN} ${REPO}/${FILE} | |
else | |
continue | |
fi | |
# could similarly wrap Rhino or Node... | |
EXIT_CODE=$((${EXIT_CODE} + $?)) | |
done | |
# End CssLint hook | |
# | |
# Common | |
# | |
if [[ ${EXIT_CODE} -ne 0 ]]; then | |
echo "" | |
echo "Problems were found" | |
echo "Commit aborted." | |
exit ${EXIT_CODE} | |
fi |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@xbx - Thanks for this! How could I modify this file so that a file like
.env
is skipped too, in addition to the non-ASCIIs?