From c38d50eb4e425d0d042b6cc4b1b022de1f3c4bea Mon Sep 17 00:00:00 2001
From: Francois Kritzinger <francois@codesynthesis.com>
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(-)

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