Skip to content

Commit ddd7740

Browse files
committed
Fix s3_reports_path: unbound variable
1 parent 0738108 commit ddd7740

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

bin/generate-report-s3.sh

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,8 @@ fi
2727
# Resolve RESULTS_PATH to absolute path and validate it's within workspace
2828
RESULTS_PATH_ABS=$(cd "$RESULTS_PATH" && pwd -P)
2929
WORKSPACE_PATH=$(pwd -P)
30+
s3_reports_path="s3://a8c-jetpack-e2e-reports/reports"
31+
REPORTS_BASE_URL=$(node "$SCRIPT_PATH/get-config-value.js" reportDeepUrl)
3032

3133
# Ensure RESULTS_PATH doesn't escape the workspace (GitHub Actions workspace)
3234
if [[ "$RESULTS_PATH_ABS" != "$WORKSPACE_PATH"* ]] && [[ "$RESULTS_PATH_ABS" != "$(dirname "$WORKSPACE_PATH")"* ]]; then
@@ -105,9 +107,6 @@ for d in "$RESULTS_PATH"/*; do
105107
fi
106108
done
107109

108-
s3_reports_path="s3://a8c-jetpack-e2e-reports/reports"
109-
REPORTS_BASE_URL=$(node "$SCRIPT_PATH/get-config-value.js" reportDeepUrl)
110-
111110
echo "----------------------------------------"
112111

113112
for d in "$LOCAL_REPORTS_PATH"/*; do

0 commit comments

Comments
 (0)