From 69fb0a4a288af1aea7dcecc295f50eaccbdce70a Mon Sep 17 00:00:00 2001 From: QuantumGhost Date: Mon, 17 Mar 2025 19:28:25 +0800 Subject: [PATCH] chore: use POSIX shell syntax in pre-commit script (#16025) --- web/.husky/pre-commit | 20 ++++++++++++-------- 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/web/.husky/pre-commit b/web/.husky/pre-commit index a0d7df1687..9b78debadc 100755 --- a/web/.husky/pre-commit +++ b/web/.husky/pre-commit @@ -9,13 +9,17 @@ web_modified=false for file in $files do - if [[ $file == "api/"* && $file == *.py ]]; then - # set api_modified flag to true - api_modified=true - elif [[ $file == "web/"* ]]; then - # set web_modified flag to true - web_modified=true - fi + # Use POSIX compliant pattern matching + case "$file" in + api/*.py) + # set api_modified flag to true + api_modified=true + ;; + web/*) + # set web_modified flag to true + web_modified=true + ;; + esac done # run linters based on the modified modules @@ -24,7 +28,7 @@ if $api_modified; then echo "Running Ruff linter on api module" # python style checks rely on `ruff` in path - if ! command -v ruff &> /dev/null; then + if ! command -v ruff > /dev/null 2>&1; then echo "Installing linting tools (Ruff, dotenv-linter ...) ..." poetry install -C api --only lint fi