yamanin/registry
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: move WORKSPACE_ARG logic inside function to avoid template variable error

Browse Source 
Fixes test failures by ensuring WORKSPACE_ARG is defined within the function
scope rather than as a global template variable.
This commit is contained in:
blink-so[bot] 2025-08-20 23:22:42 +00:00
parent 2da86abcb9
commit 56dbdd9a5e
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
registry/coder/modules/vscode-web/run.sh
View File

@ -22,15 +22,15 @@ if [ "${DISABLE_TRUST}" = true ]; then
DISABLE_TRUST_ARG="--disable-workspace-trust"
fi
# Set workspace or folder argument
WORKSPACE_ARG=""
if [ -n "${WORKSPACE}" ]; then
WORKSPACE_ARG="${WORKSPACE}"
elif [ -n "${FOLDER}" ]; then
WORKSPACE_ARG="${FOLDER}"
fi
run_vscode_web() {
# Determine what to pass as the workspace argument
WORKSPACE_ARG=""
if [ -n "${WORKSPACE}" ]; then
WORKSPACE_ARG="${WORKSPACE}"
elif [ -n "${FOLDER}" ]; then
WORKSPACE_ARG="${FOLDER}"
fi
echo "👷 Running $VSCODE_WEB serve-local with workspace: ${WORKSPACE_ARG}"
echo "Check logs at ${LOG_PATH}!"
"$VSCODE_WEB" serve-local "$EXTENSION_ARG" "$SERVER_BASE_PATH_ARG" "$DISABLE_TRUST_ARG" --port "${PORT}" --host 127.0.0.1 --accept-server-license-terms --without-connection-token --telemetry-level "${TELEMETRY_LEVEL}" "${WORKSPACE_ARG}" > "${LOG_PATH}" 2>&1 &