From 6199917a8537a3c1f06ac6345d06a283c101fc6a Mon Sep 17 00:00:00 2001 From: Francois Kritzinger Date: Thu, 24 Oct 2024 09:25:59 +0200 Subject: Pass service_data kind+phase to constructors --- mod/mod-ci-github.cxx | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) (limited to 'mod/mod-ci-github.cxx') 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). // -- cgit v1.1