diff options
| author | Francois Kritzinger <francois@codesynthesis.com> | 2024-10-24 09:25:59 +0200 |
|---|---|---|
| committer | Francois Kritzinger <francois@codesynthesis.com> | 2024-10-24 13:11:30 +0200 |
| commit | 6199917a8537a3c1f06ac6345d06a283c101fc6a (patch) | |
| tree | c3777663dcd47e2542f4d345f038b2f5fca4569a /mod/mod-ci-github.cxx | |
| parent | bc94f77ea57edfd3e5ba0d2505f451ce56bfd422 (diff) | |
Pass service_data kind+phase to constructors
Diffstat (limited to 'mod/mod-ci-github.cxx')
| -rw-r--r-- | mod/mod-ci-github.cxx | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/mod/mod-ci-github.cxx b/mod/mod-ci-github.cxx index 6289d8b..058bd48 100644 --- a/mod/mod-ci-github.cxx +++ b/mod/mod-ci-github.cxx @@ -473,15 +473,15 @@ namespace brep // string sid (cs.repository.node_id + ":" + cs.check_suite.head_sha); - // @@ Let's pass the "state" arguments explicitly in both constructors. - // service_data sd (warning_success, iat->token, iat->expires_at, cs.installation.id, move (cs.repository.node_id), - move (cs.check_suite.head_sha), - cs.action == "rerequested" /* re_request */); + service_data::local, + false /* pre_check */, + cs.action == "rerequested" /* re_request */, + move (cs.check_suite.head_sha)); // If this check suite is being re-run, replace the existing CI request if // it exists; otherwise create a new one, doing nothing if a request @@ -712,12 +712,13 @@ namespace brep iat->expires_at, pr.installation.id, move (pr.repository.node_id), + service_data::local, // @@ TODO + true /* pre_check */, + false /* re_request */, move (pr.pull_request.head_sha) /* report_sha */, move (pr.repository.clone_url), pr.pull_request.number); - assert (sd.pre_check); // @@ Get rid (once ctor changed). - // Create an unloaded CI request for the pre-check phase (during which we // wait for the PR's merge commit and behindness to become available). // |