tracing: use fields rather than interpolation

1 files changed, 5 insertions, 4 deletions

diff --git a/src/commands/meme/history.rs b/src/commands/meme/history.rs
index e3d0de3..d6ccdcf 100644
--- a/src/commands/meme/history.rs
+++ b/src/commands/meme/history.rs
@@ -107,9 +107,10 @@ pub async fn history(ctx: PoiseContext<'_>, n: Option<usize>) -> anyhow::Result<
 
     if n > CONFIG.max_hist {
         tracing::debug!(
-            "user requested more than MAX_HIST ({}) items from history",
-            CONFIG.max_hist
+            MAX_HIST = CONFIG.max_hist,
+            "user requested more than MAX_HIST items from history",
         );
+
         util::reply(ctx, "YER PUSHIN ME OVER THE FUCKIN LINE").await?;
     }
 
@@ -127,7 +128,7 @@ pub async fn history(ctx: PoiseContext<'_>, n: Option<usize>) -> anyhow::Result<
         return Ok(());
     }
 
-    tracing::info!("reporting meme history (len {})", n);
+    tracing::info!(len = n, "reporting meme history");
 
     let resp = serenity::futures::stream::iter(records.into_iter().enumerate().rev())
         .then(|(i, rec)| async move {
@@ -181,7 +182,7 @@ pub async fn history(ctx: PoiseContext<'_>, n: Option<usize>) -> anyhow::Result<
                 Err(e) => {
                     if let Some(variant) = e.downcast_ref::<DieselError>() {
                         if *variant != NotFound {
-                            tracing::error!("error encountered loading meme history: {}", e);
+                            tracing::error!(error = %e, "error encountered loading meme history");
                         }
                     }