Skip to content

Commit efd5df2

Browse files
committed
Fix test-connect-interpreter-versions ci
Allow empty executable lists with strict bash flags
1 parent 6232b59 commit efd5df2

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

Justfile

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ test chart='all':
8888

8989
test-connect-interpreter-versions:
9090
#!/usr/bin/env bash
91-
set -euxo pipefail
91+
set -euo pipefail
9292
cd ./charts/rstudio-connect && helm dependency build && cd -
9393

9494
# find the default image
@@ -102,6 +102,8 @@ test-connect-interpreter-versions:
102102

103103
for lang in "Python" "Quarto" "R"
104104
do
105+
echo "Testing $lang"
106+
105107
# print the default connect config file for local execution in ini format
106108
# print the section and grep for the Executables to find each interpreter
107109
executables=$(
@@ -110,10 +112,11 @@ test-connect-interpreter-versions:
110112
--show-only templates/configmap.yaml | \
111113
sed -n -e "/\[$lang\]/,/\[*\]/ p" | \
112114
grep Executable | awk -F= '{print $2}' | \
113-
xargs)
115+
xargs || echo "")
114116

115117
for ex in $executables
116118
do
119+
echo "Checking $ex"
117120
docker run --rm $image /bin/bash -c "command -v $ex"
118121
done
119122
done

0 commit comments

Comments
 (0)