Merge pull request #1869 from Expensify/Rory-FixUpdateStagingDeployCash

Move UpdateStagingDeployCash to after when tags are pushed
Expensify · Mar 18, 2021 · 1dc4201 · 1dc4201
2 parents 191a292 + 6afb2f1
commit 1dc4201
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/.github/workflows/preDeploy.yml b/.github/workflows/preDeploy.yml
@@ -107,20 +107,20 @@ jobs:
           body: Update version to ${{ steps.bumpVersion.outputs.NEW_VERSION }}
           labels: automerge
 
+      - name: Tag version
+        run: git tag ${{ steps.bumpVersion.outputs.NEW_VERSION }}
+
+      #TODO: Once we add cherry picking, we will need run this from elsewhere
+      - name: 🚀 Push tags to trigger staging deploy 🚀
+        run: git push --tags
+
       - name: Update StagingDeployCash
         uses: Expensify/Expensify.cash/.github/actions/createOrUpdateStagingDeploy@master
         with:
           GITHUB_TOKEN: ${{ secrets.OS_BOTIFY_TOKEN }}
           NPM_VERSION: ${{ steps.bumpVersion.outputs.NEW_VERSION }}
           NEW_PULL_REQUESTS: https://github.com/Expensify/Expensify.cash/pull/${{ needs.chooseDeployActions.outputs.mergedPullRequest }}
 
-      #TODO: Once we add cherry picking, we will need run this from elsewhere
-      - name: Tag version
-        run: git tag ${{ steps.bumpVersion.outputs.NEW_VERSION }}
-
-      - name: 🚀 Push tags to trigger staging deploy 🚀
-        run: git push --tags
-
       # This Slack step is duplicated in all workflows, if you make a change to this step, make sure to update all
       # the other workflows with the same change
       - uses: 8398a7/action-slack@v3