Fix failure in deploying starkit docs (starkit#69)

* Re-updated astropy_helpers to v2.0.10 * Fail the docs-CD pipeline if any command in deploy_docs.sh fails * Don't try to push docs when it's PR build
jaladh-singhal · Sep 25, 2019 · 3a90b81 · 3a90b81
1 parent 9bb9f4a
commit 3a90b81
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/astropy_helpers b/astropy_helpers
diff --git a/azure_pipelines/deploy_docs.sh b/azure_pipelines/deploy_docs.sh
@@ -1,4 +1,5 @@
 #!/bin/bash
+set -e   # Exit with nonzero exit code if anything fails
 
 # Build the documentation from the SOURCE_BRANCH
 # and push it to TARGET_BRANCH.
@@ -48,5 +49,7 @@ fi
 
 # Otherwise, commit and push
 git commit -m "Deploy to GitHub Pages: ${SHA}"
-git push $SSH_REPO $TARGET_BRANCH
-cd ..
+# Don't push if it's a PR build because it doesn't have access rights to do so
+if [ $BUILD_REASON != "PullRequest" ]; then
+  git push $SSH_REPO $TARGET_BRANCH
+fi