Skip to content

Commit

Permalink
Fix linting errors declaring and assignment on same line
Browse files Browse the repository at this point in the history
  • Loading branch information
CLHatch committed Feb 1, 2025
1 parent 4673093 commit 23368ce
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 4 deletions.
3 changes: 2 additions & 1 deletion .scripts/appvars_create_all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ set -Eeuo pipefail
IFS=$'\n\t'

appvars_create_all() {
local ENABLED_APPS=$(run_script 'enabled_apps')
local ENABLED_APPS
ENABLED_APPS=$(run_script 'enabled_apps')
if [[ -n ${ENABLED_APPS-} ]]; then
notice "Creating environment variables for enabled apps. Please be patient, this can take a while."
for APPNAME in ${ENABLED_APPS}; do
Expand Down
3 changes: 2 additions & 1 deletion .scripts/appvars_migrate_all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,8 @@ IFS=$'\n\t'

appvars_migrate_all() {
run_script 'appvars_migrate_enabled_lines'
local INSTALLED_APPS=$(run_script 'installed_apps')
local INSTALLED_APPS
INSTALLED_APPS=$(run_script 'installed_apps')
for APPNAME in ${INSTALLED_APPS-}; do
run_script 'appvars_migrate' "${APPNAME}"
done
Expand Down
3 changes: 2 additions & 1 deletion .scripts/appvars_purge_all.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,8 @@ set -Eeuo pipefail
IFS=$'\n\t'

appvars_purge_all() {
local DISABLED_APPS=$(run_script 'disabled_apps')
local DISABLED_APPS
DISABLED_APPS=$(run_script 'disabled_apps')
if [[ -n ${DISABLED_APPS-} ]]; then
if [[ ${CI-} == true ]] || run_script 'question_prompt' "${PROMPT:-CLI}" Y "Would you like to purge variables for all disabled apps?"; then
info "Purging disabled app variables."
Expand Down
3 changes: 2 additions & 1 deletion .scripts/yml_merge.sh
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@ yml_merge() {
run_script 'env_update'
local COMPOSE_FILE=""
notice "Adding enabled app templates to merge docker-compose.yml. Please be patient, this can take a while."
local ENABLED_APPS=$(run_script 'enabled_apps')
local ENABLED_APPS
ENABLED_APPS=$(run_script 'enabled_apps')
for APPNAME in ${ENABLED_APPS-}; do
local FILENAME=${APPNAME,,}
local APP_FOLDER="${TEMPLATES_FOLDER}/${FILENAME}"
Expand Down

0 comments on commit 23368ce

Please sign in to comment.