diff --git a/hooks/pre-commit b/hooks/pre-commit index 9b7733965dc..c6eebf4d5f9 100755 --- a/hooks/pre-commit +++ b/hooks/pre-commit @@ -40,6 +40,12 @@ echo "${reset}" echo -ne "Checking for files that need boilerplate... " files=($(git diff --cached --name-only --diff-filter ACM)) +# We always make sure there is one file in the files list. Some tools check +# the whole repo if they get no files, so in fact, this is much faster on +# git commit --amend +if [[ ${#files[@]} -eq 0 ]]; then + files+=("README.md") +fi out=($(hack/boilerplate/boilerplate.py "${files[@]}")) if [[ "${#out}" -ne 0 ]]; then echo "${red}ERROR!" @@ -54,10 +60,8 @@ else fi echo "${reset}" -allfiles=($(git diff --cached --name-only --diff-filter ACM | grep -v -e "third_party" -e "Godeps")) - echo -ne "Checking for problems with flag names... " -invalid_flag_lines=$(hack/verify-flags-underscore.py "${allfiles[@]}") +invalid_flag_lines=$(hack/verify-flags-underscore.py "${files[@]}") if [[ "${invalid_flag_lines:-}" != "" ]]; then echo "${red}ERROR!" echo "There appear to be problems with the following"