Merge pull request #3 from xiaoyuandajian/fix-#2062

Fix apache#2062
davidyuan1223 · Jan 4, 2023 · ce8cab9 · ce8cab9
2 parents e797029 + c40b5f3
commit ce8cab9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/style.yml b/.github/workflows/style.yml
@@ -103,5 +103,4 @@ jobs:
       - name: checkout files in bin
         uses: azohra/shell-linter@latest
         with:
-          path: "bin"
-          severity: "error"
+          path: "bin"
diff --git a/bin/load-kyuubi-env.sh b/bin/load-kyuubi-env.sh
@@ -59,11 +59,12 @@ fi
 
 export KYUUBI_WORK_DIR_ROOT="${KYUUBI_WORK_DIR_ROOT:-"${KYUUBI_HOME}/work"}"
 if [[ ! -e ${KYUUBI_WORK_DIR_ROOT} ]]; then
-  mkdir -p ${KYUUBI_WORK_DIR_ROOT}
+  mkdir -p "${KYUUBI_WORK_DIR_ROOT}"
 fi
 
 if [[ -z ${JAVA_HOME} ]]; then
   if [[ $(command -v java) ]]; then
+    # shellcheck disable=SC2155
     export JAVA_HOME="$(dirname $(dirname $(which java)))"
   fi
 fi