Merge pull request #6040 from shaneknapp/fixing-stupid-bash-redux-red…

…ux-redux [DH-301] fixing bash conditionals... again
berkeley-dsep-infra · Aug 28, 2024 · 37fada0 · 37fada0
2 parents 84d5248 + 42be9af
commit 37fada0
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/.github/workflows/deploy-hubs.yaml b/.github/workflows/deploy-hubs.yaml
@@ -27,7 +27,7 @@ jobs:
           # present, this means the base hub image has changed, and all hubs
           # (staging or prod) need to be redeployed.
           #
-          if [[ -n GITHUB_PR_LABEL_HUB_IMAGES || -n GITHUB_PR_LABEL_JUPYTERHUB_DEPLOYMENT ]]; then
+          if [[ -n ${GITHUB_PR_LABEL_HUB_IMAGES} || -n ${GITHUB_PR_LABEL_JUPYTERHUB_DEPLOYMENT} ]]; then
             echo "DEPLOY=1" >> $GITHUB_ENV
           # Otherwise, check to see if the PR labels contain any hubs, and 
           # deploy just those hubs to staging.
@@ -121,7 +121,7 @@ jobs:
           # present, this means the base hub image has changed, and all hubs
           # (staging or prod) need to be redeployed.
           #
-          if [[ -v GITHUB_PR_LABEL_HUB_IMAGES || -v GITHUB_PR_LABEL_JUPYTERHUB_DEPLOYMENT ]]; then
+          if [[ -n ${GITHUB_PR_LABEL_HUB_IMAGES} || -n ${GITHUB_PR_LABEL_JUPYTERHUB_DEPLOYMENT} ]]; then
             echo "DEPLOY=1" >> $GITHUB_ENV
           # Otherwise, check to see if the PR labels contain any hubs, and 
           # deploy just those hubs to prod.