From 2f9fce88b0a9324b730e868fd6a0b0b13fd8a9a9 Mon Sep 17 00:00:00 2001 From: Brandon Thomas Date: Sat, 10 Oct 2026 12:10:59 -0400 Subject: [PATCH] Review fix-up: allow the desktop logger's clock read under the new disallowed-methods lint The logger (apps/gridcraft/src/logging.rs, merged from main) reads SystemTime::now; the desktop app never runs on wasm, so allow the lint at that call. Co-Authored-By: Claude Opus 5.5 --- apps/gridcraft/src/logging.rs | 1 + 1 file changed, 1 insertion(+) diff --git a/apps/gridcraft/src/logging.rs b/apps/gridcraft/src/logging.rs index d73cd7131..e3b56a8cb 100644 --- a/apps/gridcraft/src/logging.rs +++ b/apps/gridcraft/src/logging.rs @@ -262,6 +262,7 @@ impl log::Log for AppLogger { } // Formatted before the sink is locked, so a message that logs while it is formatted can't deadlock. let thread = std::thread::current(); + #[allow(clippy::disallowed_methods)] // the desktop app (native only): wasm never runs it let line = format_line(×tamp(SystemTime::now()), record.level(), thread.name().unwrap_or("?"), record.target(), &record.args().to_string()); if self.stderr {