Skip to content

Commit f317fc7

Browse files
fix: only notify Slack when merge-train PR is dequeued without being merged (#19146)
## Summary - Skip Slack notification when a merge-train PR is dequeued due to successful merge - Only notify when the PR is removed from the merge queue for other reasons (failed checks, conflicts, etc.) ## Test plan - Merge to next, verify no notification when merge-train PR merges successfully - Verify notification still fires when merge-train PR is dequeued due to failure
2 parents 56c661d + f6e8300 commit f317fc7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/merge-queue-dequeue-notify.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ jobs:
88
notify-slack:
99
name: Notify Slack
1010
runs-on: ubuntu-latest
11-
if: startsWith(github.event.pull_request.head.ref, 'merge-train/')
11+
if: startsWith(github.event.pull_request.head.ref, 'merge-train/') && github.event.pull_request.merged != true
1212
steps:
1313
- name: Checkout
1414
uses: actions/checkout@v4

0 commit comments

Comments
 (0)