Git Pre-push钩子多检查项时仅最后一项失败才终止推送的问题

我明白你遇到的问题了——你的pre-push钩子里虽然列了多项检查，但只有最后一个命令失败时才会阻止推送，前面的检查失败根本拦不住。这是因为默认的shell脚本行为会忽略前面命令的失败状态，继续执行后续命令，而Git只看整个钩子脚本的最终退出码（0表示允许推送，非0则阻止）。

问题根源 #

你的原始钩子脚本是这样的：

# Pre-push hooks
# Lint stuff before committing
grunt jshint
eslint
tslint
# Verifies that versions and hashed value of the project’s package.json matches yarn’s lock file.
yarn check --integrity

假设grunt jshint执行失败返回非零码，但脚本会继续跑后面的eslint、tslint和yarn check，只要最后一个yarn check返回0，整个脚本就返回0，Git就允许推送了。调换顺序也没用，因为不管哪个在前，只要最后一个通过，脚本就放行。

两种可行的解决方案 #

方案1：开启脚本的“失败即终止”模式

在脚本最开头加上set -e，这样任何命令执行失败（返回非零码）时，脚本会立即终止，不再执行后续命令，同时整个脚本返回非零码，Git就会阻止推送。

修改后的脚本：

set -e

# Pre-push hooks
# Lint stuff before committing
grunt jshint
eslint
tslint
# Verifies that versions and hashed value of the project’s package.json matches yarn’s lock file.
yarn check --integrity

⚠️ 注意：set -e会让任何非零退出码的命令直接终止脚本，如果你的某些检查允许失败（但这里显然所有检查都必须通过），这个方案完全适用。

方案2：为每个检查命令显式添加失败退出逻辑

如果你不想全局开启set -e，可以给每个必须通过的检查命令后面加上|| exit 1，意思是“如果前面的命令失败，就立即退出脚本并返回1（非零码）”。

修改后的脚本：

# Pre-push hooks
# Lint stuff before committing
grunt jshint || exit 1
eslint || exit 1
tslint || exit 1
# Verifies that versions and hashed value of the project’s package.json matches yarn’s lock file.
yarn check --integrity || exit 1

这种方式更灵活，你可以针对不同的命令设置不同的失败处理逻辑（比如某些命令失败后提示信息再退出），但在你的场景下，只要任何检查失败就阻止推送，这样写就足够了。

验证修改效果 #

修改完钩子脚本后，记得给它加上可执行权限（如果还没加的话）：

chmod +x .git/hooks/pre-push

然后测试一下：故意制造一个jshint错误，执行git push，这时候脚本会在jshint失败后立即终止，Git会提示推送被阻止，完美解决你的问题。

内容的提问来源于stack exchange，提问作者megha