chore(#1474): document the too_many_arguments keeps in hive-forge + hive-ag3nt
This commit is contained in:
parent
5d0f3d060b
commit
05002acda4
3 changed files with 18 additions and 3 deletions
|
|
@ -421,7 +421,12 @@ async fn serve_main<S: Surface>(socket: &Path, poll_ms: u64) -> Result<()> {
|
||||||
/// The long-running message loop. Long-polls the broker via
|
/// The long-running message loop. Long-polls the broker via
|
||||||
/// `S::recv_next`, drives a turn per message, parks on auth-failed,
|
/// `S::recv_next`, drives a turn per message, parks on auth-failed,
|
||||||
/// otherwise retries.
|
/// otherwise retries.
|
||||||
#[allow(clippy::too_many_arguments)]
|
#[allow(
|
||||||
|
clippy::too_many_arguments,
|
||||||
|
reason = "the harness's long-lived deps threaded into one serve loop, \
|
||||||
|
wired once from main; bundling into a struct would just move the \
|
||||||
|
same fields one level out (cf. Coordinator::open)"
|
||||||
|
)]
|
||||||
async fn serve_loop<S: Surface>(
|
async fn serve_loop<S: Surface>(
|
||||||
socket: &Path,
|
socket: &Path,
|
||||||
interval: Duration,
|
interval: Duration,
|
||||||
|
|
|
||||||
|
|
@ -625,7 +625,12 @@ fn format_state_change_notification(
|
||||||
Some(out)
|
Some(out)
|
||||||
}
|
}
|
||||||
|
|
||||||
#[allow(clippy::too_many_arguments)]
|
#[allow(
|
||||||
|
clippy::too_many_arguments,
|
||||||
|
reason = "the notification poll's config + mutable subscription state, \
|
||||||
|
wired once from the poll loop; a struct would just move the \
|
||||||
|
same fields one level out"
|
||||||
|
)]
|
||||||
#[allow(
|
#[allow(
|
||||||
clippy::too_many_lines,
|
clippy::too_many_lines,
|
||||||
reason = "single-pass notification poll loop — split would obscure the \
|
reason = "single-pass notification poll loop — split would obscure the \
|
||||||
|
|
|
||||||
|
|
@ -254,7 +254,12 @@ fn print_ci(sha: &str, state: &str, statuses: &[Value]) {
|
||||||
}
|
}
|
||||||
|
|
||||||
/// Render the full PR health block.
|
/// Render the full PR health block.
|
||||||
#[allow(clippy::too_many_arguments)]
|
#[allow(
|
||||||
|
clippy::too_many_arguments,
|
||||||
|
reason = "pure display helper — the args are the already-fetched PR fields \
|
||||||
|
it prints; a struct would just mirror the API response for a \
|
||||||
|
single call site"
|
||||||
|
)]
|
||||||
fn print_pr(
|
fn print_pr(
|
||||||
pr: u64,
|
pr: u64,
|
||||||
title: &str,
|
title: &str,
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue