Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(NA): add trap for SIGINT in the git pre-commit hook #52662

Merged
merged 2 commits into from
Dec 10, 2019
Merged

feat(NA): add trap for SIGINT in the git pre-commit hook #52662

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4e4c1f3
feat(NA): add trap for SIGINT in the git precommit hook
mistic Dec 7, 2019
85196ee
Merge remote-tracking branch 'upstream/master' into add-trap-to-sigin…
mistic Dec 10, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions src/dev/register_git_hook/register_git_hook.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,15 @@ function getKbnPrecommitGitHookScript(rootPath, nodeHome, platform) {

set -euo pipefail

# Make it possible to terminate pre commit hook
# using ctrl-c so nothing else would happen or be
# sent to the output.
#
# The correct exit code on that situation
# according the linux documentation project is 130
# https://www.tldp.org/LDP/abs/html/exitcodes.html
trap "exit 130" SIGINT

has_node() {
command -v node >/dev/null 2>&1
}
Expand Down