Add in more safety around deployment (#248)

Author: patch0

.github/workflows/deploy.yml

@@ -54,10 +54,10 @@ jobs:
   setup-environment:
     runs-on: ubuntu-latest
     outputs:
-      deploy_dir: ${{ steps.deploy_dir.outputs.deploy_dir }}
-      public_url: ${{ steps.public_url.outputs.public_url }}
+      deploy_dir: ${{ steps.setup-environment.outputs.deploy_dir }}
+      public_url: ${{ steps.setup-environment.outputs.public_url }}
     steps:
-    - id: safe_ref_name
+    - id: setup-environment
       run: |
         safe_ref_name="${GITHUB_REF_NAME//[![:alnum:].]/_}"
         deploy_dir=${{inputs.prefix}}/$safe_ref_name
@@ -72,6 +72,12 @@ jobs:
       name: ${{ inputs.environment }}
       url: ${{ needs.setup-environment.outputs.public_url }}
     steps:
+
+    - name: Check deployment path
+      run: |
+        echo "Deploying to '${{ needs.setup-environment.outputs.deploy_dir }}' -- job will fail if this is empty!"
+        [ -n "${{ needs.setup-environment.outputs.deploy_dir }}" ]
+
     - name: Checkout
       uses: actions/checkout@v1
 

CHANGELOG.md

@@ -10,7 +10,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 ### Changed
 
 - Set indentation to 4 spaces to match PEP8 standard (#246)
-- Update Github workflow not to strip dots from the git ref, and remove test dependency for main and ref deploys (#244)
+- Update Github workflow not to strip dots from the git ref, and remove test dependency for main and ref deploys (#244, #248)
 
 ## [0.6.0]