From e096ef691a2fce66031a27ff29171fc2e7330fd9 Mon Sep 17 00:00:00 2001 From: Josh Date: Tue, 24 Mar 2026 09:43:28 -0400 Subject: [PATCH 1/2] fix(hooks): incorrect skipped output in mixed exec/non-exec batches The `found` variable is incorrectly decremented for non-executable scripts, which leads to misleading output when a hook folder contains an executable + a non-executable script (output will indicate the folder was fully scripted). Signed-off-by: Josh --- docker-entrypoint.sh | 1 - 1 file changed, 1 deletion(-) diff --git a/docker-entrypoint.sh b/docker-entrypoint.sh index 303aa0052..74bfa141b 100755 --- a/docker-entrypoint.sh +++ b/docker-entrypoint.sh @@ -36,7 +36,6 @@ run_path() { while read -r script_file_path; do if ! [ -x "${script_file_path}" ]; then echo "==> The script \"${script_file_path}\" was skipped, because it lacks the executable flag" - found=$((found-1)) continue fi From dbd693dc67dc564e4ea0b382f9e36c337f547690 Mon Sep 17 00:00:00 2001 From: Josh Date: Fri, 27 Mar 2026 10:01:19 -0400 Subject: [PATCH 2/2] chore(hooks): make skipped/completed logic more intuitive Signed-off-by: Josh --- docker-entrypoint.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/docker-entrypoint.sh b/docker-entrypoint.sh index 74bfa141b..efaceeb3a 100755 --- a/docker-entrypoint.sh +++ b/docker-entrypoint.sh @@ -50,10 +50,10 @@ run_path() { echo "==> Finished executing the script: \"${script_file_path}\"" done - if [ "$found" -lt "1" ]; then - echo "==> Skipped: the \"$1\" folder does not contain any valid scripts" + if [ "$found" -gt "0" ]; then + echo "=> Completed executing scripts in the \"$1\" folder" else - echo "=> Completed executing scripts in the \"$1\" folder" + echo "==> Skipped: the \"$1\" folder does not contain any valid scripts" fi ) }