Lint deployment script

Urban-Analytics-Technology-Platform · Jul 2, 2024 · 764ef06 · 764ef06
1 parent 22aa741
commit 764ef06
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/deploy.sh b/deploy.sh
@@ -13,16 +13,18 @@ fi
 
 export IGNORE_EXPERIMENTAL_WARNINGS=1
 export DAGSTER_MODULE_NAME=popgetter
-export DAGSTER_HOME=$(mktemp -d)
-touch $DAGSTER_HOME/dagster.yaml  # Silences Dagster warnings
+DAGSTER_HOME=$(mktemp -d)
+export DAGSTER_HOME
+touch "$DAGSTER_HOME"/dagster.yaml  # Silences Dagster warnings
 
 echo "Relevant environment variables:"
 echo "  - POPGETTER_COUNTRIES: $POPGETTER_COUNTRIES"
 echo "  - ENV: $ENV"
-if [ $ENV == "prod" ]; then
+if [ "$ENV" == "prod" ]; then
     export AZURE_STORAGE_ACCOUNT=popgetter
     export AZURE_CONTAINER=prod
-    export AZURE_DIRECTORY=$(python -c 'import popgetter; print(popgetter.__version__)' 2>/dev/null)
+    AZURE_DIRECTORY=$(python -c 'import popgetter; print(popgetter.__version__)' 2>/dev/null)
+    export AZURE_DIRECTORY
     if [ -z "$SAS_TOKEN" ]; then
         echo "SAS_TOKEN environment variable not set; it is required for Azure deployments"
         exit 1