This is an automated email from the ASF dual-hosted git repository.
jihoonson pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/master by this push:
new d89d4ff Git hooks should fail on errors; pass args to git hooks
(#12322)
d89d4ff is described below
commit d89d4ff58894a69eda1e584628acec8eafe93168
Author: Jihoon Son <[email protected]>
AuthorDate: Thu Mar 10 09:07:50 2022 +0900
Git hooks should fail on errors; pass args to git hooks (#12322)
* Git hooks should fail on errors
* don't set shell to pass args
---
hooks/run-all-in-dir.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hooks/run-all-in-dir.py b/hooks/run-all-in-dir.py
index 6922e04..d0f2f59 100755
--- a/hooks/run-all-in-dir.py
+++ b/hooks/run-all-in-dir.py
@@ -31,4 +31,4 @@ for hook in os.listdir(hooks_dir):
if not hook.startswith("_"):
command = [os.path.join(hooks_dir, hook)] + args
print("Running {}".format(command))
- subprocess.run(command, shell=True)
+ subprocess.run(command, check=True)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
