aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFrancois Kritzinger <francois@codesynthesis.com>2024-05-02 14:43:27 +0200
committerFrancois Kritzinger <francois@codesynthesis.com>2024-06-05 09:12:46 +0200
commit77f9de9c742aff1d9d55534d5f0736edc0c8c631 (patch)
tree23341df61b2cde6a2c64ebf8ddb27c6b310a7df5
parent71d2f0ecf58f52cf37eaa987309c7ec64dbab6f6 (diff)
Post-review changes
-rw-r--r--mod/mod-ci-github-gq.cxx6
-rw-r--r--mod/mod-ci-github.cxx2
2 files changed, 6 insertions, 2 deletions
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)))
{