From 9c780e2c55db7dc9482df6528710768152533f60 Mon Sep 17 00:00:00 2001 From: Francois Kritzinger Date: Thu, 14 Mar 2024 16:37:29 +0200 Subject: Post-review changes --- mod/mod-ci-github.cxx | 29 +++++++++++------------------ 1 file changed, 11 insertions(+), 18 deletions(-) (limited to 'mod/mod-ci-github.cxx') diff --git a/mod/mod-ci-github.cxx b/mod/mod-ci-github.cxx index 385b370..d51a9f7 100644 --- a/mod/mod-ci-github.cxx +++ b/mod/mod-ci-github.cxx @@ -419,9 +419,15 @@ namespace brep string al ("cr" + to_string (i)); // Field alias. - // @@ TODO: revise. + // Check run name. // - string nm (b.package_name.string () + '/' + b.target_config_name); // Name + string nm (b.package_name.string () + '/' + + b.package_version.string () + '/' + + b.target.string () + '/' + + b.target_config_name + '/' + + b.package_config_name + '/' + + b.toolchain_name + '/' + + b.toolchain_version.string ()); os << gq_name (al) << ":createCheckRun(input: {" << '\n' << " name: " << gq_str (nm) << ',' << '\n' @@ -445,19 +451,6 @@ namespace brep return os.str (); } - // Serialize an "update check run" for a build to JSON for the REST API. - // - // @@ TMP We're going to have to do the check run updates using the REST API - // because the GraphQL API check run updates take only an ID (which - // we would have to store) whereas the REST version takes a name. - // - // static string - // rest_check_run (uint64_t ri, // Repository ID - // const string& hs, // Head SHA - // const build& b) - // { - // } - bool ci_github:: handle_check_suite_request (check_suite_event cs) { @@ -508,9 +501,9 @@ namespace brep "clang", brep::version ("4.5.6")); - cout << gq_check_runs (cs.repository.node_id, - cs.check_suite.head_sha, - builds) + cout << queue_check_runs (cs.repository.node_id, + cs.check_suite.head_sha, + builds) << endl; return true; -- cgit v1.1