From 648f0ef8ed7369f2d513db3ca2942a11196ba370 Mon Sep 17 00:00:00 2001 From: Mrinal Kalakrishnan Date: Wed, 14 Jan 2026 18:29:03 -0800 Subject: [PATCH] Fix git-lfs check to display error message The previous code used `git lfs &>/dev/null` followed by checking `$?`, but since the script uses `set -e`, the script would exit immediately on the non-zero return code before reaching the error message. This also fixes the typo "insalled" -> "installed". Fixes #151, #167, #176 Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering) Co-Authored-By: Claude Co-Authored-By: Happy --- scripts/run_dev.sh | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/scripts/run_dev.sh b/scripts/run_dev.sh index 86f08e54..d521ea2e 100755 --- a/scripts/run_dev.sh +++ b/scripts/run_dev.sh @@ -138,9 +138,8 @@ if [[ -z "$(docker ps)" ]] ; then fi # Check if git-lfs is installed. -git lfs &>/dev/null -if [[ $? -ne 0 ]] ; then - print_error "git-lfs is not insalled. Please make sure git-lfs is installed before you clone the repo." +if ! git lfs &>/dev/null; then + print_error "git-lfs is not installed. Please make sure git-lfs is installed before you clone the repo." exit 1 fi