Chore: refactor publish workflow for sqlglotrs releasing

tobymao · May 4, 2024 · 0deffd8 · 0deffd8
1 parent 6c61ebb
commit 0deffd8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/.github/workflows/python-publish.yml b/.github/workflows/python-publish.yml
@@ -17,7 +17,9 @@ jobs:
       - uses: actions/checkout@v3
       - id: check_deploy
         name: Check if sqlglotrs should be deployed
-        run: bash ./.github/workflows/should_deploy_sqlglotrs.sh
+        run: |
+          bash ./.github/workflows/should_deploy_sqlglotrs.sh
+          if [ $? -eq 0 ]; then exit 1; fi
 
   build-rs:
     needs: should-deploy-rs

diff --git a/.github/workflows/should_deploy_sqlglotrs.sh b/.github/workflows/should_deploy_sqlglotrs.sh
@@ -8,4 +8,8 @@ PREVIOUS_TAG=$(git tag --sort=-creatordate | head -n 2 | tail -n 1)
 PREVIOUS_TAG_COMMIT=$(git rev-list -n 1 $PREVIOUS_TAG)
 
 # We should only deploy sqlglotrs if Cargo.toml was modified between this tag and the previous one
-exit $(git merge-base --is-ancestor $PREVIOUS_TAG_COMMIT $CARGO_TOML_MOST_RECENT_COMMIT)
+SHOULD_DEPLOY=$(git merge-base --is-ancestor $PREVIOUS_TAG_COMMIT $CARGO_TOML_MOST_RECENT_COMMIT)
+
+# Print the SHOULD_DEPLOY variable for debugging purposes
+echo "SHOULD_DEPLOY: $SHOULD_DEPLOY"
+exit $SHOULD_DEPLOY