server: fix clippy warning

The clippy::uninlined_format_args warning in 1.67 was downgraded to
pedantic in 1.67.1 due to lack of support in rust-analyzer, so we're not
updating that one yet.

app/src/lib.rs

@@ -1,6 +1,7 @@
 #![recursion_limit = "256"]
 #![forbid(non_ascii_idents)]
-#![allow(clippy::nonstandard_macro_braces)]
+#![allow(clippy::uninlined_format_args)]
+
 pub mod components;
 pub mod infra;
 

migration-tool/src/main.rs

@@ -1,3 +1,5 @@
+#![allow(clippy::uninlined_format_args)]
+
 use std::collections::HashSet;
 
 use anyhow::{anyhow, Result};

server/src/domain/sql_migrations.rs

@@ -336,15 +336,14 @@ pub async fn migrate_from_version(
     pool: &DbConnection,
     version: SchemaVersion,
 ) -> anyhow::Result<()> {
-    if version > LAST_SCHEMA_VERSION {
-        anyhow::bail!("DB version downgrading is not supported");
-    } else if version == LAST_SCHEMA_VERSION {
-        return Ok(());
+    match version.cmp(&LAST_SCHEMA_VERSION) {
+        std::cmp::Ordering::Less => info!(
+            "Upgrading DB schema from {} to {}",
+            version.0, LAST_SCHEMA_VERSION.0
+        ),
+        std::cmp::Ordering::Equal => return Ok(()),
+        std::cmp::Ordering::Greater => anyhow::bail!("DB version downgrading is not supported"),
     }
-    info!(
-        "Upgrading DB schema from {} to {}",
-        version.0, LAST_SCHEMA_VERSION.0
-    );
     let builder = pool.get_database_backend();
     if version < SchemaVersion(2) {
         // Drop the not_null constraint on display_name. Due to Sqlite, this is more complicated:

server/src/main.rs

@@ -1,6 +1,7 @@
 #![forbid(unsafe_code)]
 #![forbid(non_ascii_idents)]
-#![allow(clippy::nonstandard_macro_braces)]
+// TODO: Remove next line once ubuntu upgrades rustc to >=1.67.1
+#![allow(clippy::uninlined_format_args)]
 
 use std::time::Duration;