Merge pull request #686 from bashly-framework/update/var-order

Move global variable declarations after environment variable declarations

Author: DannyBen

lib/bashly/views/command/initialize.gtx

@@ -6,8 +6,8 @@
 
 if root_command?
   > 
-  = render(:variables).indent(2)
   = render(:environment_variables_default).indent 2
+  = render(:variables).indent(2)
 end
 
 if user_file_exist?('initialize')

spec/approvals/fixtures/var-order

@@ -0,0 +1,11 @@
++ bundle exec bashly generate
+creating user files in src
+skipped src/root_command.sh (exists)
+created ./cli
+run ./cli --help to test your bash script
++ ./cli
+args: none
+
+environment variables:
+- $CLI_CONFIG_FILE = config.yaml
+$config_file == $CLI_CONFIG_FILE == config.yaml

spec/fixtures/workspaces/var-order/.gitignore

@@ -0,0 +1 @@
+cli

spec/fixtures/workspaces/var-order/README.md

@@ -0,0 +1,3 @@
+This fixture ensures that environment variables are declared and populated
+begore variables defined in bashly.yml, to allow using the values from them
+in the variables.

spec/fixtures/workspaces/var-order/src/bashly.yml

@@ -0,0 +1,12 @@
+name: cli
+help: Ensure env var / var order
+version: 0.1.0
+
+environment_variables:
+- name: cli_config_file
+  help: Path to config file
+  default: config.yaml
+
+variables:
+- name: config_file
+  value: $CLI_CONFIG_FILE

spec/fixtures/workspaces/var-order/src/root_command.sh

@@ -0,0 +1,2 @@
+inspect_args
+echo "\$config_file == \$CLI_CONFIG_FILE == $config_file"

spec/fixtures/workspaces/var-order/test.sh

@@ -0,0 +1,6 @@
+#!/usr/bin/env bash
+
+set -x
+
+bundle exec bashly generate
+./cli