From 525214e9c8b118fdb83e9b8cc28b9e52980f940b Mon Sep 17 00:00:00 2001 From: Markus Pesch Date: Sun, 16 Aug 2026 21:11:27 +0200 Subject: [PATCH] fix(install-yq): ensure version check failure aborts script Move the yq binary version check out of the echo command substitution into a standalone variable assignment. With set -e, a failed command inside $() as an argument to echo does not trigger errexit, silently swallowing errors like "Exec format error". A separate assignment correctly propagates the failure. Co-authored-by: Copilot --- install-yq.sh | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/install-yq.sh b/install-yq.sh index 9af5212..bde5879 100755 --- a/install-yq.sh +++ b/install-yq.sh @@ -72,7 +72,9 @@ else fi chmod +x "${bin_path}" -echo "Installed yq version $(${bin_path} --version) in ${bin_path}" + +yq_installed_version="$("${bin_path}" --version)" +echo "Installed ${yq_installed_version} in ${bin_path}" # Ensure yq is in PATH for subsequent steps if [[ -n "${GITHUB_PATH:-}" ]]; then