Merge pull request #634 from DannyBen/fix/validate_not_empty

Fix premature exit in `validate_not_empty` validation function

Author: DannyBen

examples/colors-usage/src/lib/colors.sh

@@ -10,10 +10,21 @@
 ## Color output will be disabled if `NO_COLOR` environment variable is set
 ## in compliance with https://no-color.org/
 ##
+## In case you wish to enable auto detection for color output based on the
+## terminal being interactive (TTY), call `enable_auto_colors` in your 
+## `src/initialize.sh` (Run `bashly add hooks` to add this file).
+##
+enable_auto_colors() {
+  ## If NO_COLOR has not been set and stdout is not a TTY, disable colors
+  if [[ -z ${NO_COLOR+x} && ! -t 1 ]]; then
+    NO_COLOR=1
+  fi
+}
+
 print_in_color() {
   local color="$1"
   shift
-  if [[ -z ${NO_COLOR+x} ]]; then
+  if [[ "${NO_COLOR:-}" == "" ]]; then
     printf "$color%b\e[0m\n" "$*"
   else
     printf "%b\n" "$*"
@@ -26,17 +37,26 @@ yellow() { print_in_color "\e[33m" "$*"; }
 blue() { print_in_color "\e[34m" "$*"; }
 magenta() { print_in_color "\e[35m" "$*"; }
 cyan() { print_in_color "\e[36m" "$*"; }
+black() { print_in_color "\e[30m" "$*"; }
+white() { print_in_color "\e[37m" "$*"; }
+
 bold() { print_in_color "\e[1m" "$*"; }
 underlined() { print_in_color "\e[4m" "$*"; }
+
 red_bold() { print_in_color "\e[1;31m" "$*"; }
 green_bold() { print_in_color "\e[1;32m" "$*"; }
 yellow_bold() { print_in_color "\e[1;33m" "$*"; }
 blue_bold() { print_in_color "\e[1;34m" "$*"; }
 magenta_bold() { print_in_color "\e[1;35m" "$*"; }
 cyan_bold() { print_in_color "\e[1;36m" "$*"; }
+black_bold() { print_in_color "\e[1;30m" "$*"; }
+white_bold() { print_in_color "\e[1;37m" "$*"; }
+
 red_underlined() { print_in_color "\e[4;31m" "$*"; }
 green_underlined() { print_in_color "\e[4;32m" "$*"; }
 yellow_underlined() { print_in_color "\e[4;33m" "$*"; }
 blue_underlined() { print_in_color "\e[4;34m" "$*"; }
 magenta_underlined() { print_in_color "\e[4;35m" "$*"; }
 cyan_underlined() { print_in_color "\e[4;36m" "$*"; }
+black_underlined() { print_in_color "\e[4;30m" "$*"; }
+white_underlined() { print_in_color "\e[4;37m" "$*"; }

examples/dependencies-alt/README.md

@@ -52,6 +52,16 @@ commands:
 ### `$ ./cli download`
 
 ````shell
+# This file is located at 'src/download_command.sh'.
+# It contains the implementation for the 'cli download' command.
+# The code you write here will be wrapped by a function named 'cli_download_command()'.
+# Feel free to edit this file; your changes will persist when regenerating.
+args: none
+
+deps:
+- ${deps[git]} = /usr/bin/git
+- ${deps[http_client]} = /usr/bin/curl
+- ${deps[ruby]} = /home/vagrant/.rbenv/versions/3.4.1/bin/ruby
 
 
 ````

examples/render-mandoc/README.md

@@ -102,7 +102,7 @@ ISSUE TRACKER
 AUTHORS
        Lana Lang.
 
-Version 0.1.0                     April 2025                       download(1)
+Version 0.1.0                      May 2025                        download(1)
 
 
 ````

examples/render-mandoc/docs/download.1

@@ -1,6 +1,6 @@
 .\" Automatically generated by Pandoc 3.2
 .\"
-.TH "download" "1" "May 2025" "Version 0.1.0" "Sample application"
+.TH "download" "1" "June 2025" "Version 0.1.0" "Sample application"
 .SH NAME
 \f[B]download\f[R] \- Sample application
 .SH SYNOPSIS

examples/render-mandoc/docs/download.md

@@ -1,6 +1,6 @@
 % download(1) Version 0.1.0 | Sample application
 % Lana Lang
-% May 2025
+% June 2025
 
 NAME
 ==================================================

examples/validations/README.md

@@ -52,13 +52,23 @@ commands:
     validate: file_exists
 
 - name: build
+  help: Build the project
   environment_variables:
   - name: build_dir
     help: Path to the build directory
     default: release
 
     # Validations also work on environment variables
     validate: dir_exists
+
+- name: deploy
+  help: Deploy to production
+  flags:
+  - long: --user
+    help: Username
+    arg: username
+    required: true
+    validate: not_empty
 ````
 
 
@@ -116,17 +126,31 @@ must be an existing directory
 
 ````
 
-### `$ BUILD_DIR=src ./validate build`
+### `$ BUILD_DIR=src ./validate build\`
 
 ````shell
-# This file is located at 'src/build_command.sh'.
-# It contains the implementation for the 'validate build' command.
-# The code you write here will be wrapped by a function named 'validate_build_command()'.
-# Feel free to edit this file; your changes will persist when regenerating.
-args: none
 
-environment variables:
-- $BUILD_DIR = src
+
+````
+
+### `$ ./validate deploy --user ''`
+
+````shell
+validation error in --user USERNAME:
+must not be empty
+
+
+````
+
+### `$ ./validate deploy --user admin`
+
+````shell
+# This file is located at 'src/deploy_command.sh'.
+# It contains the implementation for the 'validate deploy' command.
+# The code you write here will be wrapped by a function named 'validate_deploy_command()'.
+# Feel free to edit this file; your changes will persist when regenerating.
+args:
+- ${args[--user]} = admin
 
 
 ````

examples/validations/src/bashly.yml

@@ -28,10 +28,20 @@ commands:
     validate: file_exists
 
 - name: build
+  help: Build the project
   environment_variables:
   - name: build_dir
     help: Path to the build directory
     default: release
 
     # Validations also work on environment variables
     validate: dir_exists
+
+- name: deploy
+  help: Deploy to production
+  flags:
+  - long: --user
+    help: Username
+    arg: username
+    required: true
+    validate: not_empty

examples/validations/src/deploy_command.sh

@@ -0,0 +1,5 @@
+echo "# This file is located at 'src/deploy_command.sh'."
+echo "# It contains the implementation for the 'validate deploy' command."
+echo "# The code you write here will be wrapped by a function named 'validate_deploy_command()'."
+echo "# Feel free to edit this file; your changes will persist when regenerating."
+inspect_args

examples/validations/src/lib/validations/validate_dir_exists.sh

@@ -1,4 +1,6 @@
 ## [@bashly-upgrade validations]
 validate_dir_exists() {
-  [[ -d "$1" ]] || echo "must be an existing directory"
+  if [[ ! -d "$1" ]]; then
+    echo "must be an existing directory"
+  fi
 }

examples/validations/src/lib/validations/validate_file_exists.sh

@@ -1,4 +1,6 @@
 ## [@bashly-upgrade validations]
 validate_file_exists() {
-  [[ -f "$1" ]] || echo "must be an existing file"
+  if [[ ! -f "$1" ]]; then
+    echo "must be an existing file"
+  fi
 }