From 5935b03afd4d42e3264586fb64c783dcfaac11ec Mon Sep 17 00:00:00 2001 From: Michael McLoughlin Date: Sun, 5 Mar 2023 20:32:25 -0800 Subject: [PATCH] ci: skip entire autoland job for non-bot users (#382) In contributor PRs such as #372 we see that the pr/autoland job fails due to permissions issues obtaining an app token from a fork. This job is actually only required to auto-land bot PRs. Instead of skipping just the auto-land setting step, we should be skipping the entire thing. Fixes #381 --- .github/workflows/pr.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml index bf98e0ee..cffb3889 100644 --- a/.github/workflows/pr.yml +++ b/.github/workflows/pr.yml @@ -5,6 +5,7 @@ on: pull_request: jobs: automerge: + if: github.actor == 'cadobot[bot]' && !github.event.pull_request.draft runs-on: ubuntu-latest steps: - name: Generate Bot Token @@ -14,7 +15,6 @@ jobs: app_id: ${{ secrets.APP_ID }} private_key: ${{ secrets.APP_PRIVATE_KEY }} - name: Automerge Bot Pull Requests - if: github.actor == 'cadobot[bot]' && !github.event.pull_request.draft run: gh pr merge --auto --squash "${{ github.event.pull_request.html_url }}" env: GITHUB_TOKEN: ${{ steps.bot.outputs.token }}