Skip to content

Commit 38f555e

Browse files
committed
chore: fix lint issues in update-vscode
1 parent 169b51e commit 38f555e

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

ci/dev/update-vscode.sh

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,10 @@ main() {
1515

1616
# Ask which version we should update to
1717
# In the future, we'll automate this and grab the latest version automatically
18-
read -p "What version of VSCode would you like to update to? (i.e. 1.52) " VSCODE_VERSION_TO_UPDATE
18+
read -r -p "What version of VSCode would you like to update to? (i.e. 1.52) " VSCODE_VERSION_TO_UPDATE
1919

2020
# Check that this version exists
21-
if [[ -z $(git ls-remote --heads vscode release/$VSCODE_VERSION_TO_UPDATE) ]]; then
21+
if [[ -z $(git ls-remote --heads vscode release/"$VSCODE_VERSION_TO_UPDATE") ]]; then
2222
echo "Oops, that doesn't look like a valid version."
2323
echo "You entered: $VSCODE_VERSION_TO_UPDATE"
2424
echo "Verify that this branches exists here: https://github.com/microsoft/vscode/branches/all?query=release%2F$VSCODE_VERSION_TO_UPDATE"
@@ -39,10 +39,10 @@ main() {
3939
# If we don't do this, the opening a draft PR step won't work
4040
# because it will stop and ask where you want to push the branch
4141
CURRENT_BRANCH=$(git branch --show-current)
42-
if [[ -z $(git ls-remote --heads origin $CURRENT_BRANCH) ]]; then
42+
if [[ -z $(git ls-remote --heads origin "$CURRENT_BRANCH") ]]; then
4343
echo "Doesn't look like you've pushed this branch to remote"
4444
echo -e "Pushing now using: git push origin $CURRENT_BRANCH\n"
45-
git push origin $CURRENT_BRANCH
45+
git push origin "$CURRENT_BRANCH"
4646
fi
4747

4848
echo "Opening a draft PR on GitHub"
@@ -52,7 +52,7 @@ main() {
5252
echo "Going to try to update vscode for you..."
5353
echo -e "Running: git subtree pull --prefix lib/vscode vscode release/${VSCODE_VERSION_TO_UPDATE} --squash\n"
5454
# Try to run subtree update command
55-
git subtree pull --prefix lib/vscode vscode release/${VSCODE_VERSION_TO_UPDATE} --squash --message "chore(vscode): update to $VSCODE_VERSION_TO_UPDATE"
55+
git subtree pull --prefix lib/vscode vscode release/"${VSCODE_VERSION_TO_UPDATE}" --squash --message "chore(vscode): update to $VSCODE_VERSION_TO_UPDATE"
5656
}
5757

5858
main "$@"

0 commit comments

Comments
 (0)