From 0bda8b04545412f158bd151c23d5de9ce3e79460 Mon Sep 17 00:00:00 2001 From: daanpersoons <143718186+daanpersoons@users.noreply.github.com> Date: Thu, 6 Aug 2026 10:39:32 +0200 Subject: [PATCH] infra-k8s-preview: no-op when no k8s paths changed Empty changed_paths ran one empty loop iteration (here-string), so touch diff-new/ failed on PRs touching no k8s/. Early-exit + skip empty paths. --- .github/workflows/infra-k8s-preview.yml | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/.github/workflows/infra-k8s-preview.yml b/.github/workflows/infra-k8s-preview.yml index 64bbff1..576367d 100644 --- a/.github/workflows/infra-k8s-preview.yml +++ b/.github/workflows/infra-k8s-preview.yml @@ -36,8 +36,14 @@ jobs: run: | changed_paths=`echo $matrix | jq -r '.[].path'` + if [ -z "$changed_paths" ]; then + echo "No changed k8s paths; nothing to preview." + exit 0 + fi + # Generate new content while IFS= read -r path; do + [ -z "$path" ] && continue echo "Processing new $path…" mkdir -p `dirname diff-new/$path` if [ -e "$path" ]; then @@ -55,6 +61,7 @@ jobs: # Generate old content while IFS= read -r path; do + [ -z "$path" ] && continue echo "Processing old $path…" mkdir -p `dirname diff-old/$path` if [ -e "$path" ]; then @@ -69,6 +76,7 @@ jobs: { echo 'contents<$path" echo -e '\n```diff' diff -u "diff-old/$path" "diff-new/$path" || true