aboutsummaryrefslogtreecommitdiff
path: root/mod/mod-ci-github.cxx
diff options
context:
space:
mode:
Diffstat (limited to 'mod/mod-ci-github.cxx')
-rw-r--r--mod/mod-ci-github.cxx12
1 files changed, 12 insertions, 0 deletions
diff --git a/mod/mod-ci-github.cxx b/mod/mod-ci-github.cxx
index 0ddc75d..d829ff6 100644
--- a/mod/mod-ci-github.cxx
+++ b/mod/mod-ci-github.cxx
@@ -1579,7 +1579,10 @@ namespace brep
if (cr.state == build_state::built)
{
if (conclusion)
+ {
+ assert (cr.status);
*conclusion |= *cr.status;
+ }
}
else
conclusion = nullopt;
@@ -1798,6 +1801,15 @@ namespace brep
// result_status into a gq_ function because converting to a GitHub
// conclusion/title/summary is reasonably complicated.
//
+ // @@@ We need to redo that code:
+ //
+ // - Pass the vector of check runs with new state (and status) set.
+ // - Update synchronized flag inside those functions.
+ // - Update the state to built if it's already built on GitHub --
+ // but then what do we set the status to?
+ // - Maybe signal in return value (optional<bool>?) that there is
+ // a discrepancy.
+ //
cr.status = b.status;
// Update the conclusion check run if all check runs are now built.