Only emit terminal codes to terminal

dannysauer · dannysauer · commit 65891e23f341 · 2022-10-13T13:23:13.000-05:00
Check for presence of tput and for stdout being a terminal before emitting terminal control codes.
Remove unnecessary dependence on `say`
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 ] && which tput >/dev/null && echo -n "$(tput setaf 2)✓$(tput sgr 0) "
+  echo "Installed ${hook} hook to ${dest}"
 }
 
 install_all_hooks() {

Original file line number	Diff line number	Diff line change
`@@ -205,7 +205,8 @@ install_hook() {`
`205`	`205`	`echo "#!/usr/bin/env bash" > "${dest}"`
`206`	`206`	`echo "git secrets --${cmd} -- \"\$@\"" >> "${dest}"`
`207`	`207`	`chmod +x "${dest}"`
`208`		`- say "$(tput setaf 2)✓$(tput sgr 0) Installed ${hook} hook to ${dest}"`
	`208`	`+ [ -t 1 ] && which tput >/dev/null && echo -n "$(tput setaf 2)✓$(tput sgr 0) "`
	`209`	`+ echo "Installed ${hook} hook to ${dest}"`
`209`	`210`	`}`
`210`	`211`
`211`	`212`	`install_all_hooks() {`