Merge pull request #221 from dannysauer/bug/no-require-terminal

Only emit terminal codes to terminal (and don't require `say`)
awslabs · Jun 20, 2023 · 0ab55f8 · 0ab55f8
2 parents b1fdfe0 + b9e40d8
commit 0ab55f8
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/git-secrets b/git-secrets
@@ -205,7 +205,8 @@ install_hook() {
   echo "#!/usr/bin/env bash" > "${dest}"
   echo "git secrets --${cmd} -- \"\$@\"" >> "${dest}"
   chmod +x "${dest}"
-  say "$(tput setaf 2)✓$(tput sgr 0) Installed ${hook} hook to ${dest}"
+  [ -t 1 ] && command -v tput &> /dev/null && echo -n "$(tput setaf 2)✓$(tput sgr 0) "
+  echo "Installed ${hook} hook to ${dest}"
 }
 
 install_all_hooks() {