Merge pull request #648 from dxw/fix-setup-messaging

Fix setup script messaging
dxw · Nov 24, 2023 · 16dae4b · 16dae4b
2 parents 5688145 + c87373d
commit 16dae4b
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/script/no-docker/setup b/script/no-docker/setup
@@ -22,7 +22,7 @@ bundle exec rails db:drop
 DB_DROP_RESULT=$?
 set -e
 
-if [ -z "$CI" ] && [ -n "$DB_DROP_RESULT" ]; then
+if [ -z "$CI" ] && [ "$DB_DROP_RESULT" != 0 ]; then
   printf "\\nDatabase drop failed. Continue anyway? [y/N] "
   read -r
 
@@ -44,7 +44,7 @@ RAILS_ENV="test" bundle exec rails db:drop
 DB_DROP_RESULT=$?
 set -e
 
-if [ -z "$CI" ] && [ -n "$DB_DROP_RESULT" ]; then
+if [ -z "$CI" ] && [ "$DB_DROP_RESULT" != 0 ]; then
   printf "\\nDatabase drop failed. Continue anyway? [y/N] "
   read -r