From abf0489e7477d5e8fc21c2c96cc2f24daf3b1586 Mon Sep 17 00:00:00 2001 From: Tao Zhou Date: Mon, 3 Feb 2020 14:34:58 +0100 Subject: [PATCH] Apply linter pre-commit hook only to Added/Copied/Modified it will fail on deleted/renamed files. Change-Id: I4285f0f9ccc4f2b6c875816c4df25a15fd20222b --- tools/dev-hooks/pre-commit | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/dev-hooks/pre-commit b/tools/dev-hooks/pre-commit index b77f38279f..af87b7e3ac 100755 --- a/tools/dev-hooks/pre-commit +++ b/tools/dev-hooks/pre-commit @@ -31,7 +31,7 @@ gitroot=${gitroot:-.}; eslint=${gitroot}/node_modules/eslint/bin/eslint.js # Run eslint over changed frontend code -CHANGED_UI_FILES=$(git diff --cached --name-only -- '*.js' '*.html' | grep 'polygerrit-ui') && true +CHANGED_UI_FILES=$(git diff --cached --name-only --diff-filter=ACM -- '*.js' '*.html' | grep 'polygerrit-ui') && true if [ "${CHANGED_UI_FILES}" ]; then if $eslint --fix ${CHANGED_UI_FILES}; then # Add again in case lint fix modified some files