Skip to content

Commit 183fd73

Browse files
committed
github: don't self-request pr review
1 parent 211f96e commit 183fd73

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/github.ml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -149,8 +149,8 @@ let get_project_owners (pr : pull_request) ({ rules } : Config_t.project_owners)
149149
with Re2.Exceptions.Regex_match_failed _ -> First reviewer
150150
)
151151
|> fun (reviewers, team_reviewers) ->
152-
let already_requested = List.map ~f:(fun r -> r.login) pr.requested_reviewers in
152+
let already_requested_or_author = pr.user.login :: List.map ~f:(fun r -> r.login) pr.requested_reviewers in
153153
let already_requested_team = List.map ~f:(fun r -> r.slug) pr.requested_teams in
154-
let reviewers = List.filter ~f:(not $ List.mem already_requested ~equal:String.equal) reviewers in
154+
let reviewers = List.filter ~f:(not $ List.mem already_requested_or_author ~equal:String.equal) reviewers in
155155
let team_reviewers = List.filter ~f:(not $ List.mem already_requested_team ~equal:String.equal) team_reviewers in
156156
if List.is_empty reviewers && List.is_empty team_reviewers then None else Some { reviewers; team_reviewers }

0 commit comments

Comments
 (0)