From 2e9ae54977d8e874065f48b3b9578fd713f1d395 Mon Sep 17 00:00:00 2001 From: Seyed Yahya Shirazi Date: Thu, 4 Jun 2026 23:32:56 -0700 Subject: [PATCH] fix(ci): ignore bot comments in the community merge command The App's confirmation comment contains 'LGTM / merge' and re-triggered the issue_comment workflow (which then no-op'd: bot is not a maintainer). Skip comments where comment.user.type == 'Bot' so the App's own comment no longer spawns an extra run. Closes #302 --- .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?