Review exceptions thrown by github-ci API functions

1 files changed, 8 insertions, 0 deletions

diff --git a/mod/mod-ci-github-gq.hxx b/mod/mod-ci-github-gq.hxx
index 7c564d7..86ab859 100644
--- a/mod/mod-ci-github-gq.hxx
+++ b/mod/mod-ci-github-gq.hxx
@@ -24,6 +24,8 @@ namespace brep
   // `check_runs` with the new data (node id and state_synced). Return false
   // and issue diagnostics if the request failed.
   //
+  // Throw invalid_argument.
+  //
   // Note that creating a check_run named `foo` will effectively replace any
   // existing check_runs with that name. They will still exist on the GitHub
   // servers but GitHub will only consider the latest one (for display in the
@@ -40,6 +42,8 @@ namespace brep
   // data (node id, state, and state_synced). Return false and issue
   // diagnostics if the request failed.
   //
+  // Throw invalid_argument.
+  //
   // If the details_url is absent GitHub will use the app's homepage.
   //
   // The gq_built_result is required if the build_state is built because
@@ -66,6 +70,8 @@ namespace brep
   // Update a check run on GitHub. Update `cr` with the new data (state and
   // state_synced). Return false and issue diagnostics if the request failed.
   //
+  // Throw invalid_argument.
+  //
   // Note that GitHub allows any state transitions except from built (but
   // built to built is allowed). The latter case is signalled by setting the
   // check_run state_synced member to false and the state member to built.
@@ -99,6 +105,8 @@ namespace brep
   // Issue diagnostics and return absent if the request failed (which means it
   // will be treated by the caller as still being generated).
   //
+  // Throw invalid_argument if the node id is invalid.
+  //
   // Note that the first request causes GitHub to start preparing the test
   // merge commit.
   //