Update on "ci: Clean up logic for merge -r"

seemethere · seemethere · commit c5cef686c207 · 2023-11-21T14:16:10.000-06:00
Rely on built in bash conditionals for doing the if statement rather than relying on $? To avoid issues observed in #111008 (comment) Signed-off-by: Eli Uriegas <eliuriegasmeta.com> [ghstack-poisoned]
diff --git a/.github/workflows/trymerge.yml b/.github/workflows/trymerge.yml
@@ -33,6 +33,7 @@ jobs:
           git config --global user.email "pytorchmergebot@users.noreply.github.com"
           git config --global user.name "PyTorch MergeBot"
       - name: Merge PR
+        shell: bash
         env:
           GITHUB_TOKEN: ${{ secrets.MERGEBOT_TOKEN }}
           PR_NUM: ${{ github.event.client_payload.pr_num }}
