From 37ae72c19df06208104547b14fafbc9c139de2ce Mon Sep 17 00:00:00 2001 From: atlas Date: Sun, 26 Jul 2026 19:41:34 +0200 Subject: [PATCH] docs(hive-forge): add the missing # Errors section on fetch_combined_in --- hive-forge/src/verbs/pr_status.rs | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/hive-forge/src/verbs/pr_status.rs b/hive-forge/src/verbs/pr_status.rs index 4d943ff2..dc1f35d8 100644 --- a/hive-forge/src/verbs/pr_status.rs +++ b/hive-forge/src/verbs/pr_status.rs @@ -165,6 +165,13 @@ fn pr_status(client: &Client, repo: &str, pr: u64) -> Result<()> { /// Fetch the combined commit status for `sha` in the client's repo. /// Goes through the raw-JSON escape hatch rather than the typed client /// so an empty (no-CI) state stays reportable — see [`CombinedStatus`]. +/// +/// # Errors +/// +/// Returns an error if `repo` isn't a well-formed `owner/name`, or if +/// the status GET fails (transport, non-2xx, or a body that isn't even +/// loosely the expected shape). An empty state is NOT an error — that's +/// the case this function exists to report. pub(crate) fn fetch_combined_in(client: &Client, repo: &str, sha: &str) -> Result { let (owner, name) = crate::client::split_repo(repo)?; client.get_api_json(&format!("/repos/{owner}/{name}/commits/{sha}/status"), &[])