Skip to content
This repository was archived by the owner on May 15, 2025. It is now read-only.

fix(vscode-web): remove exit if extension installation fails #318

Merged
merged 5 commits into from
Oct 15, 2024
Merged

fix(vscode-web): remove exit if extension installation fails #318

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
072538d
fix(vscode-web): remove exit if extension installation fails.
matifali Oct 14, 2024
0b8f130
fix conditions
matifali Oct 15, 2024
421b08a
Merge branch 'main' into atif/vscode-web-exit
matifali Oct 15, 2024
a352e33
Fix indentation in vscode-web run script
matifali Oct 15, 2024
ff7ee03
Update vscode-web module version to 1.0.20
matifali Oct 15, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions vscode-web/run.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,14 +72,12 @@ for extension in "$${EXTENSIONLIST[@]}"; do
output=$($VSCODE_WEB "$EXTENSION_ARG" --install-extension "$extension" --force)
if [ $? -ne 0 ]; then
echo "Failed to install extension: $extension: $output"
exit 1
fi
done

if [ "${AUTO_INSTALL_EXTENSIONS}" = true ]; then
if ! command -v jq > /dev/null; then
echo "jq is required to install extensions from a workspace file."
Copy link
Member

@code-asher code-asher Oct 14, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh wait we still need some kind of exit/bypass here. Maybe put the rest of this block under an else?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alternatively, we use node from the code-server install instead of jq, since we know that will exist.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess it is ok to just give the message without exiting so that the script can continue.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well we have to put an else (or keep the exit) otherwise it will continue and try to use jq, which will fail

Copy link
Member

@code-asher code-asher Oct 14, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let me know if I misunderstood though 😆

if [ "${AUTO_INSTALL_EXTENSIONS}" = true ]; then
  if ! command -v jq > /dev/null; then
    echo "jq is required to install extensions from a workspace file."
  else
   WORKSPACE_DIR="$HOME"
   if [ -n "${FOLDER}" ]; then
     WORKSPACE_DIR="${FOLDER}"
   fi

   if [ -f "$WORKSPACE_DIR/.vscode/extensions.json" ]; then
     printf "🧩 Installing extensions from %s/.vscode/extensions.json...\n" "$WORKSPACE_DIR"
     extensions=$(jq -r '.recommendations[]' "$WORKSPACE_DIR"/.vscode/extensions.json)
     for extension in $extensions; do
       $VSCODE_WEB "$EXTENSION_ARG" --install-extension "$extension" --force
     done
   fi
  fi
fi

exit 0
fi

WORKSPACE_DIR="$HOME"
Expand Down
Loading