From aa1cda2b393345de89f7c77007c5dd5026b09dac Mon Sep 17 00:00:00 2001 From: Evie Gauthier Date: Thu, 9 Jul 2026 10:01:31 -0400 Subject: [PATCH] =?UTF-8?q?ci:=20review-gate=20=E2=80=94=20exempt=20bot-au?= =?UTF-8?q?thored=20PRs=20(e.g.=20dependabot)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Seer and Codex don't review bot-authored PRs, so once `Review bots done` is a required check, every dependabot dependency-bump PR fail-closes on the timeout and can never merge. Skip the wait (job passes) when the PR author is a Bot, alongside the existing merge_group/draft no-ops. Human PRs still gate on Seer + Codex. Co-Authored-By: Claude Opus 4.8 (1M context) Claude-Session: https://claude.ai/code/session_01DaLybCc9v3px6dL2j9pF6W --- .github/workflows/review-gate.yml | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/.github/workflows/review-gate.yml b/.github/workflows/review-gate.yml index 8e35c5f8..a49f7823 100644 --- a/.github/workflows/review-gate.yml +++ b/.github/workflows/review-gate.yml @@ -43,9 +43,14 @@ jobs: runs-on: ubuntu-latest steps: - name: Wait for review bots to finish on the current head - # No-op pass on merge_group (see header) and on draft PRs (which can't - # merge and which the bots skip anyway). Marking a PR ready re-triggers. - if: github.event_name == 'pull_request' && github.event.pull_request.draft == false + # No-op pass on merge_group (see header), on draft PRs (which can't merge + # and which the bots skip anyway; marking ready re-triggers), and on + # bot-authored PRs (e.g. dependabot) — Seer and Codex don't review those, + # so waiting would fail-closed and block every automated dependency bump. + if: >- + github.event_name == 'pull_request' + && github.event.pull_request.draft == false + && github.event.pull_request.user.type != 'Bot' env: GH_TOKEN: ${{ github.token }} REPO: ${{ github.repository }}