[7.x] return 1 when precommit_hook or local version fail (#400… (#40089)

elastic · Jul 1, 2019 · ee43c6e · ee43c6e
1 parent 30cce18
commit ee43c6e
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/dev/register_git_hook/register_git_hook.js b/src/dev/register_git_hook/register_git_hook.js
@@ -111,11 +111,12 @@ function getKbnPrecommitGitHookScript(rootPath, nodeHome, platform) {
   }
 
   execute_precommit_hook() {
-    node scripts/precommit_hook
+    node scripts/precommit_hook || return 1
+
     PRECOMMIT_FILE="./.git/hooks/pre-commit.local"
     if [ -x "\${PRECOMMIT_FILE}" ]; then
       echo "Executing local precommit hook found in \${PRECOMMIT_FILE}"
-      "$PRECOMMIT_FILE"
+      "$PRECOMMIT_FILE" || return 1
     fi
   }