Skip to content

Commit 3b99392

Browse files
committed
github: request project owner review if non-draft only
1 parent 85161a1 commit 3b99392

File tree

1 file changed

+14
-8
lines changed

1 file changed

+14
-8
lines changed

lib/action.ml

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -210,16 +210,22 @@ module Action (Github_api : Api.Github) (Slack_api : Api.Slack) = struct
210210

211211
let do_github_tasks ctx (req : Github.t) =
212212
let cfg = Context.get_config_exn ctx in
213+
let project_owners (pull_request : pull_request) repository number =
214+
match Github.get_project_owners pull_request.labels cfg.project_owners with
215+
| Some reviewers ->
216+
( match%lwt Github_api.request_reviewers ~ctx ~repo:repository ~number ~reviewers with
217+
| Ok () -> Lwt.return_unit
218+
| Error e -> action_error e
219+
)
220+
| None -> Lwt.return_unit
221+
in
213222
match req with
214-
| Github.Pull_request { action = Labeled; pull_request; repository; number; _ } ->
223+
| Github.Pull_request
224+
{ action; pull_request = { draft = false; state = Open; _ } as pull_request; repository; number; _ } ->
215225
begin
216-
match Github.get_project_owners pull_request.labels cfg.project_owners with
217-
| Some reviewers ->
218-
( match%lwt Github_api.request_reviewers ~ctx ~repo:repository ~number ~reviewers with
219-
| Ok () -> Lwt.return_unit
220-
| Error e -> action_error e
221-
)
222-
| None -> Lwt.return_unit
226+
match action with
227+
| Ready_for_review | Labeled -> project_owners pull_request repository number
228+
| _ -> Lwt.return_unit
223229
end
224230
| _ -> Lwt.return_unit
225231

0 commit comments

Comments
 (0)