From a53ba167d69c81d9f7935f6fd9b2ddc1aa85cba9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gabriel=20Fr=C3=B3es=20Franco?= Date: Sun, 31 May 2026 08:13:30 -0300 Subject: [PATCH] fix(lsp): silence clippy too_many_arguments on render_page_html Adding list_margin pushed render_page_html to 8 params, tripping clippy::too_many_arguments under -D warnings in CI. Every parameter is a distinct render input and the function has a single production caller, so scope an #[allow] rather than introduce an options-struct abstraction. Co-Authored-By: Claude Opus 4.7 --- crates/nuwiki-lsp/src/export.rs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/crates/nuwiki-lsp/src/export.rs b/crates/nuwiki-lsp/src/export.rs index ac7a545..6b5bfa5 100644 --- a/crates/nuwiki-lsp/src/export.rs +++ b/crates/nuwiki-lsp/src/export.rs @@ -193,6 +193,9 @@ pub fn is_excluded(patterns: &[String], name: &str) -> bool { /// When set, it is stripped from wiki/interwiki link targets so a link /// written with the extension (`[[todo.wiki]]`) exports to `todo.html`, /// matching how the LSP resolves the same link for in-editor navigation. +// Every parameter is a distinct, cohesive render input; bundling them into a +// struct would add an abstraction without simplifying the single caller. +#[allow(clippy::too_many_arguments)] pub fn render_page_html( doc: &DocumentNode, template: Option,