diff options
author | Francois Kritzinger <francois@codesynthesis.com> | 2024-03-14 16:37:29 +0200 |
---|---|---|
committer | Francois Kritzinger <francois@codesynthesis.com> | 2024-10-15 09:05:27 +0200 |
commit | 9c780e2c55db7dc9482df6528710768152533f60 (patch) | |
tree | 9db7d8c88d20567aace2327b739a038b73667623 /mod/mod-ci-github.cxx | |
parent | ee11338f1d83a06e5a531d392494e47bed41f06f (diff) |
Post-review changes
Diffstat (limited to 'mod/mod-ci-github.cxx')
-rw-r--r-- | mod/mod-ci-github.cxx | 29 |
1 files changed, 11 insertions, 18 deletions
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; |