Skip to content

Commit 6d62ad9

Browse files
authored
Merge pull request #25808 from dvdksn/codex/dedupe-bot-pr-reviews
ci: deduplicate automated PR reviews
2 parents 05b9013 + 8d61d5b commit 6d62ad9

2 files changed

Lines changed: 10 additions & 2 deletions

File tree

.github/workflows/pr-review-trigger.yml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,34 @@
11
name: PR Review - Trigger
22
on:
33
pull_request:
4-
types: [ready_for_review, opened, review_requested]
4+
types: [opened, review_requested]
55
pull_request_review_comment:
66
types: [created]
77

88
permissions: {}
99

1010
jobs:
1111
save-context:
12+
if: >
13+
github.event_name != 'pull_request' ||
14+
github.event.action != 'review_requested' ||
15+
github.event.requested_reviewer.login == 'docker-agent'
1216
runs-on: ubuntu-latest
1317
steps:
1418
- name: Save event context
1519
env:
1620
PR_NUMBER: ${{ github.event.pull_request.number }}
1721
PR_HEAD_SHA: ${{ github.event.pull_request.head.sha }}
22+
REQUESTED_REVIEWER: ${{ github.event.requested_reviewer.login }}
1823
COMMENT_JSON: ${{ toJSON(github.event.comment) }}
1924
run: |
2025
mkdir -p context
2126
printf '%s' "${{ github.event_name }}" > context/event_name.txt
2227
printf '%s' "$PR_NUMBER" > context/pr_number.txt
2328
printf '%s' "$PR_HEAD_SHA" > context/pr_head_sha.txt
29+
if [ "${{ github.event_name }}" = "pull_request" ]; then
30+
printf '%s' "$REQUESTED_REVIEWER" > context/requested_reviewer.txt
31+
fi
2432
if [ "${{ github.event_name }}" = "pull_request_review_comment" ]; then
2533
printf '%s' "$COMMENT_JSON" > context/comment.json
2634
fi

.github/workflows/pr-review.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ permissions:
1111

1212
jobs:
1313
review:
14-
uses: docker/docker-agent-action/.github/workflows/review-pr.yml@e96a4bb40cac114f64358621e1d08346c8eadc8c # v2.0.1
14+
uses: docker/docker-agent-action/.github/workflows/review-pr.yml@baf90543d81f5de59751dfd10e6cf45e21a5a982 # v2.0.3
1515
# Scoped to the job so other jobs in this workflow aren't over-permissioned
1616
permissions:
1717
contents: read # Read repository files and PR diffs

0 commit comments

Comments
 (0)