From 912c94d987d3b2bd7500469090873a1575c4dbfb Mon Sep 17 00:00:00 2001
From: Francois Kritzinger <francois@codesynthesis.com>
Date: Thu, 2 May 2024 14:43:27 +0200
Subject: Post-review changes

---
 mod/mod-ci-github-gq.cxx | 6 +++++-
 mod/mod-ci-github.cxx    | 2 +-
 2 files changed, 6 insertions(+), 2 deletions(-)

(limited to 'mod')

diff --git a/mod/mod-ci-github-gq.cxx b/mod/mod-ci-github-gq.cxx
index 8e436e9..4a4f56d 100644
--- a/mod/mod-ci-github-gq.cxx
+++ b/mod/mod-ci-github-gq.cxx
@@ -494,10 +494,14 @@ namespace brep
                        build_state st,
                        optional<gq_built_result> br)
   {
-    // Must have a result if state is built. @@ du?
+    // Must have a result if state is built.
     //
     assert (st != build_state::built || br);
 
+    // Must have a details URL for building and built.
+    //
+    assert (!du.empty ());
+
     string rq (
       gq_serialize_request (
         gq_mutation_update_check_run (rid,
diff --git a/mod/mod-ci-github.cxx b/mod/mod-ci-github.cxx
index 9c084ba..dc21e0e 100644
--- a/mod/mod-ci-github.cxx
+++ b/mod/mod-ci-github.cxx
@@ -869,7 +869,7 @@ namespace brep
                                  iat->token,
                                  sd.repository_id,
                                  *cr.node_id,
-                                 "", // Don't update details_url again. @@
+                                 details_url (b),
                                  build_state::built,
                                  move (br)))
         {
-- 
cgit v1.1