Run shellcheck from action (#112)

This prevents new failing checks from being committed.

Also see #102.

.github/workflows/test.yml

@@ -209,3 +209,19 @@ jobs:
       - name: Assert installation path is ~/bin
         run: ./setup-cli/assert/path.sh ~/bin/databricks
         shell: bash
+
+  shellcheck:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          path: ./setup-cli
+
+      - name: Install shellcheck
+        run: sudo apt-get install shellcheck
+        shell: bash
+
+      - name: Run shellcheck
+        run: shellcheck ./setup-cli/*.sh
+        shell: bash

install.sh

@@ -88,7 +88,7 @@ unzip -q "${FILE}.zip"
 # Add databricks to path.
 chmod +x ./databricks
 cp ./databricks "$TARGET"
-echo "Installed $($TARGET/databricks -v) at $TARGET/databricks."
+echo "Installed $("$TARGET/databricks" -v) at $TARGET/databricks."
 
 # Clean up temporary directory.
 cd "$OLDPWD"

setup_release.sh

@@ -9,7 +9,7 @@ fi
 
 # Pull latest version from VERSION file if not set.
 if [ -z "${VERSION:-}" ]; then
-    VERSION=$(cat $(dirname "$0")/VERSION)
+    VERSION=$(cat "$(dirname "$0")"/VERSION)
 fi
 
 FILE="databricks_cli_$VERSION"