fix(shell): double quote params (#824)

.husky/commit-msg

@@ -1,4 +1,6 @@
 #!/bin/sh
-. "$(dirname $0)/_/husky.sh"
 
-npx --no-install commitlint --edit $1
+# shellcheck source=./_/husky.sh
+. "$(dirname "$0")/_/husky.sh"
+
+npx --no-install commitlint --edit "$1"

.husky/pre-commit

@@ -1,4 +1,6 @@
 #!/bin/sh
-. "$(dirname $0)/_/husky.sh"
+
+# shellcheck source=./_/husky.sh
+. "$(dirname "$0")/_/husky.sh"
 
 npm test

scripts/pre-commit

@@ -4,7 +4,7 @@ echo "husky: installing Git hooks"
 
 mkdir -p "$husky_dir/_"
 echo "_" > "$husky_dir/.gitignore"
-cp "$(dirname $0)/husky.sh" "$husky_dir/_"
+cp "$(dirname "$0")/husky.sh" "$husky_dir/_"
 git config core.hooksPath "$husky_dir"
 
 echo "husky: ok"

src/commands/add.ts

@@ -15,7 +15,7 @@ function createHookFile(file: string, cmd: string) {
     throw new Error(`${file} already exists`)
   }
 
-  const data = ['#!/bin/sh', '. "$(dirname $0)/_/husky.sh"', '', cmd].join('\n')
+  const data = ['#!/bin/sh', '. "$(dirname "$0")/_/husky.sh"', '', cmd].join('\n')
 
   fs.writeFileSync(file, data, 'utf-8')
   // Show "./file" instead of just "file"

test/config-dir.sh

@@ -1,6 +1,9 @@
-. $(dirname $0)/_functions.sh
+# shellcheck shell=bash
 
-# Example: 
+# shellcheck source=./_functions.sh
+. "$(dirname "$0")/_functions.sh"
+
+# Example:
 # .config/husky
 title "config directory"
 tempDir="/tmp/husky-config-dir-test"

test/default.sh

@@ -1,4 +1,7 @@
-. $(dirname $0)/_functions.sh
+# shellcheck shell=bash
+
+# shellcheck source=./_functions.sh
+. "$(dirname "$0")/_functions.sh"
 
 title "default"
 tempDir="/tmp/husky-default-test"
@@ -23,4 +26,3 @@ git commit -m "should fail" || echo -e "\e[0;32mOK\e[m"
 # Uninstall
 npx --no-install husky uninstall
 git config core.hooksPath || echo -e "\e[0;32mOK\e[m"
-

test/sub-dir.sh

@@ -1,4 +1,4 @@
-. $(dirname $0)/_functions.sh
+. $(dirname "$0")/_functions.sh
 
 # Example:
 # .git