diff options
| author | Gaëtan Gilbert | 2019-07-09 12:52:09 +0200 |
|---|---|---|
| committer | Gaëtan Gilbert | 2019-07-09 12:52:09 +0200 |
| commit | 876f767ef55585e4ed186db3f1d8589cbd4605c1 (patch) | |
| tree | 6e18afeb183aedecb642959336b4f9efca8d8587 /dev | |
| parent | 35d8ca72adcc3ce04cb2919c4a2d60ea0c73d24c (diff) | |
merge-pr.sh: filter reviews to remove the PR author
This removes spurious Ack-by lines
Diffstat (limited to 'dev')
| -rwxr-xr-x | dev/tools/merge-pr.sh | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/dev/tools/merge-pr.sh b/dev/tools/merge-pr.sh index 425f21de70..c0a3eeb11c 100755 --- a/dev/tools/merge-pr.sh +++ b/dev/tools/merge-pr.sh @@ -209,7 +209,8 @@ has_state() { [ "$(jq -rc 'map(select(.user.login == "'"$1"'") | .state) | any(. == "'"$2"'")' <<< "$reviews")" = true ] } -for reviewer in $(jq -rc 'map(.user.login) | unique | join(" ")' <<< "$reviews" ); do +author=$(echo "$PRDATA" | jq -rc '.user.login') +for reviewer in $(jq -rc 'map(.user.login | select(. != "'"$author"'")) | unique | join(" ")' <<< "$reviews" ); do if has_state "$reviewer" APPROVED; then msg=$(printf '%s\n' "$msg" | git interpret-trailers --trailer Reviewed-by="$reviewer") elif has_state "$reviewer" COMMENTED; then |