From 793544e0484eb663b66d0ae4d85adaf366ab4789 Mon Sep 17 00:00:00 2001 From: Seyed Yahya Shirazi Date: Thu, 4 Jun 2026 23:36:14 -0700 Subject: [PATCH] ci: sync bot-comment guard to main (#303) Keep the registered (default-branch) merge workflow in sync: ignore bot comments so the App's confirmation no longer self-triggers it. --- .github/workflows/community-admin-pr-merge.yml | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/.github/workflows/community-admin-pr-merge.yml b/.github/workflows/community-admin-pr-merge.yml index 8e96c31..48b5a58 100644 --- a/.github/workflows/community-admin-pr-merge.yml +++ b/.github/workflows/community-admin-pr-merge.yml @@ -50,8 +50,12 @@ concurrency: jobs: merge-on-command: - # Only comments on pull requests (issue_comment also fires on plain issues). - if: github.event.issue.pull_request != null + # Only human comments on pull requests. issue_comment also fires on plain + # issues (filtered out), and on the App's own confirmation comment (which + # mentions "LGTM / merge") -- skipping Bot authors stops that self-trigger. + if: >- + github.event.issue.pull_request != null && + github.event.comment.user.type != 'Bot' runs-on: ubuntu-latest steps: # Cheap first gate: does the comment carry the "LGTM ... merge" command?