scriptoriumadmin/fabro
1
0
Fork 0
mirror of https://github.com/fabro-sh/fabro.git synced 2026-10-10 03:30:59 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

Inline the blob_hash_from_response alias

Browse source 
It was a one-line passthrough to parse_blob_ref with a single caller,
leaving two names for the same operation; every other consumer calls
parse_blob_ref directly.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
Scott Werner 2026-08-16 11:04:05 -04:00
parent 3524cd76d4
commit 46d4a1e5c8
1 changed files with 2 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
lib/apps/fabro-cli/src/commands/run/output.rs
View file

@ -5,7 +5,7 @@ use anyhow::{Context as _, Result};
use cli_table::format::{Border, Justify, Separator};
use cli_table::{Cell, CellStruct, Style, Table};
use fabro_api::types;
use fabro_types::{BlobHash, PullRequestLink, RunId, StageId, parse_blob_ref};
use fabro_types::{PullRequestLink, RunId, StageId, parse_blob_ref};
use fabro_util::check_report::{CheckDetail, CheckReport, CheckResult, CheckSection, CheckStatus};
use fabro_util::error::render_with_causes;
use fabro_util::printer::Printer;
@ -325,7 +325,7 @@ async fn resolve_response_string(
run_id: &RunId,
response: &str,
) -> Result<Option<String>> {
let Some(blob_hash) = blob_hash_from_response(response) else {
let Some(blob_hash) = parse_blob_ref(response) else {
return Ok(Some(response.to_string()));
};
@ -341,10 +341,6 @@ async fn resolve_response_string(
}))
}
fn blob_hash_from_response(response: &str) -> Option<BlobHash> {
parse_blob_ref(response)
}
async fn list_artifact_display_entries_with_client(
client: &server_client::Client,
run_id: &RunId,