From adfc3a8101cf362a59c4d3565539a44ec00d9d42 Mon Sep 17 00:00:00 2001 From: Jose Diaz-Gonzalez Date: Fri, 4 Sep 2015 00:18:41 -0400 Subject: [PATCH] Use dokku_log_fail in favor of dokku_log_warn If a command will exit because of invalid input, that isn't a warning, it's a failure. --- plugins/apps/commands | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/plugins/apps/commands b/plugins/apps/commands index 55fdfb89a..47a1b304f 100755 --- a/plugins/apps/commands +++ b/plugins/apps/commands @@ -10,7 +10,7 @@ case "$1" in apps:create) [[ -z $2 ]] && dokku_log_fail "Please specify an app to run the command on" - [[ -d "$DOKKU_ROOT/$2" ]] && dokku_log_warn "Name is already taken" && exit 1 + [[ -d "$DOKKU_ROOT/$2" ]] && dokku_log_fail "Name is already taken" APP="$2" mkdir -p "$DOKKU_ROOT/$APP" @@ -32,8 +32,7 @@ case "$1" in read -p "> " app_name if [[ "$app_name" != "$APP" ]]; then - dokku_log_warn "Confirmation did not match $APP. Aborted." - exit 1 + dokku_log_fail "Confirmation did not match $APP. Aborted." fi fi