From f9d7a587a23a964be76883f1bd6ef2eee08e756c Mon Sep 17 00:00:00 2001 From: Laria Carolin Chabowski Date: Sun, 26 Apr 2020 21:50:04 +0200 Subject: Add pre-commit hooks to check for syntax errors and coding style Just run the git-hooks/install.sh script to install the pre-commit hook --- git-hooks/hooks/pre-commit | 52 ++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 52 insertions(+) create mode 100755 git-hooks/hooks/pre-commit (limited to 'git-hooks/hooks') diff --git a/git-hooks/hooks/pre-commit b/git-hooks/hooks/pre-commit new file mode 100755 index 0000000..5666e93 --- /dev/null +++ b/git-hooks/hooks/pre-commit @@ -0,0 +1,52 @@ +#!/bin/sh + +set -e + +gitdir="$(git rev-parse --git-dir)" + +changed_files() { + git diff --name-status --cached HEAD | tr '\011' '\012' | awk ' + /^[ADMTUXB]/ { n = 1; next } + /^[CR]/ { n = 2; next } + n > 0 { + print $0 + n-- + } + ' +} + +staged_file_hash() { + git ls-files -s "$1" | cut -d' ' -f2 +} + +lint_php() { + php -l >/dev/null +} + +lint_php_cs_fixer() { + php-cs-fixer fix --config="$gitdir/../.php_cs.dist" -v --dry-run --using-cache=no --show-progress=none - >&2 +} + +lint_changes() { + linter_name="$1" + grep_pattern="$2" + linter_func="$3" + + failcount="$(changed_files | grep "$grep_pattern" | while read -r f; do + hash="$(staged_file_hash "$f")" + if [ -n "$hash" ]; then + if ! git cat-file blob "$hash" | "$linter_func"; then + echo >&2 + echo "Linter $linter_name reported error for: $f" >&2 + echo >&2 + + echo failed + fi + fi + done | wc -l)" + + [ "$failcount" -eq 0 ] || return 1 +} + +lint_changes "php -l" '\.php$' lint_php || exit 1 +lint_changes "php_cs_fixer" '\.php$' lint_php_cs_fixer || exit 1 -- cgit v1.2.3-70-g09d2