chore: fix clippy string interpolation errors

A recent update to clippy introduced a new lint for strings that don't
interpolate variables. Fix any offending lines.

Signed-off-by: Ivin Joel Abraham <[email protected]>

Author: ivinjabraham

src/commands.rs

@@ -84,7 +84,7 @@ async fn set_log_level(ctx: Context<'_>, level: String) -> Result<(), Error> {
     };
 
     if reload_handle.reload(EnvFilter::new(&new_filter)).is_ok() {
-        ctx.say(format!("Log level changed to **{}**", new_filter))
+        ctx.say(format!("Log level changed to **{new_filter}**"))
             .await?;
         info!("Log level changed to {}", new_filter);
     } else {

src/graphql/queries.rs

@@ -228,14 +228,13 @@ pub async fn fetch_attendance() -> anyhow::Result<Vec<AttendanceRecord>> {
     let query = format!(
         r#"
         query {{
-            attendanceByDate(date: "{}") {{
+            attendanceByDate(date: "{today}") {{
                 name,
                 year,
                 isPresent,
                 timeIn,
             }}
-        }}"#,
-        today
+        }}"#
     );
 
     let response = client

src/tasks/lab_attendance.rs

@@ -97,7 +97,7 @@ async fn send_lab_closed_message(ctx: SerenityContext) -> anyhow::Result<()> {
         .unwrap_or_else(|| bot_user.default_avatar_url());
 
     let embed = CreateEmbed::new()
-        .title(format!("Presense Report - {}", today_date))
+        .title(format!("Presense Report - {today_date}"))
         .url(TITLE_URL)
         .author(
             CreateEmbedAuthor::new("amD")
@@ -156,7 +156,7 @@ async fn send_attendance_report(
     description.push_str(&format_attendance_list("Late", &late_list));
 
     let embed = CreateEmbed::new()
-        .title(format!("Presense Report - {}", today_date))
+        .title(format!("Presense Report - {today_date}"))
         .url(TITLE_URL)
         .author(
             CreateEmbedAuthor::new("amD")
@@ -191,15 +191,15 @@ fn format_attendance_list(title: &str, list: &[AttendanceRecord]) -> String {
         }
     }
 
-    let mut result = format!("# {}\n", title);
+    let mut result = format!("# {title}\n");
 
     for year in 1..=3 {
         if let Some(names) = by_year.get(&year) {
             if !names.is_empty() {
-                result.push_str(&format!("### Year {}\n", year));
+                result.push_str(&format!("### Year {year}\n"));
 
                 for name in names {
-                    result.push_str(&format!("- {}\n", name));
+                    result.push_str(&format!("- {name}\n"));
                 }
                 result.push('\n');
             }

src/tasks/status_update.rs

@@ -159,10 +159,7 @@ fn categorize_members(
             nice_list.push(member.clone());
         } else {
             let track = member.track.clone();
-            naughty_list
-                .entry(track)
-                .or_insert_with(Vec::new)
-                .push(member.clone());
+            naughty_list.entry(track).or_default().push(member.clone());
         }
     }
 
@@ -196,15 +193,11 @@ async fn generate_embed(
 
     description.push_str("# Leaderboard Updates\n");
 
-    description.push_str(&format!(
-        "## All-Time High Streak: {} days\n",
-        all_time_high
-    ));
+    description.push_str(&format!("## All-Time High Streak: {all_time_high} days\n"));
     description.push_str(&format_members(&all_time_high_members));
 
     description.push_str(&format!(
-        "## Current Highest Streak: {} days\n",
-        current_highest
+        "## Current Highest Streak: {current_highest} days\n"
     ));
     description.push_str(&format_members(&current_highest_members));
 
@@ -229,7 +222,7 @@ fn format_members(members: &[Member]) -> String {
             .collect::<Vec<_>>()
             .join("\n");
 
-        format!("{}\n", list)
+        format!("{list}\n")
     } else {
         String::from("More than five members hold this record!\n")
     }
@@ -239,8 +232,8 @@ fn format_defaulters(naughty_list: &GroupedMember) -> String {
     let mut description = String::new();
     for (track, missed_members) in naughty_list {
         match track {
-            Some(t) => description.push_str(&format!("## Track - {}\n", t)),
-            None => description.push_str(&format!("## Unassigned")),
+            Some(t) => description.push_str(&format!("## Track - {t}\n")),
+            None => description.push_str("## Unassigned"),
         }
 
         for member in missed_members {