Apply suggestions from code review

Co-authored-by: thesimplekid <tsk@thesimplekid.com>

crates/cdk-sql-common/src/common.rs

@@ -21,13 +21,14 @@ pub async fn migrate<C: DatabaseExecutor>(
 
     // Apply each migration if it hasn’t been applied yet
     for (name, sql) in migrations {
-        let basename = if let Some((prefix, basename)) = name.split_once(['/', '\\']) {
-            if prefix != db_prefix {
-                continue;
+        let basename = match name.split_once(['/', '\\']) {
+            Some((prefix, basename)) => {
+                if prefix != db_prefix {
+                    continue;
+                }
+                basename
             }
-            basename
-        } else {
-            name
+            None => name,
         };
 
         let is_missing = query("SELECT name FROM migrations WHERE name = :name")?

crates/cdk-sql-common/src/pool.rs

@@ -37,7 +37,7 @@ pub trait ResourceManager: Debug {
 
     /// Creates a new resource with a given config.
     ///
-    /// If `stale` is every set to TRUE it is assumed the resource is no longer valid and it will be
+    /// If `stale` is ever set to TRUE it is assumed the resource is no longer valid and it will be
     /// dropped.
     fn new_resource(
         config: &Self::Config,

crates/cdk-sql-common/src/stmt.rs

@@ -176,9 +176,11 @@ impl Statement {
         } else {
             let parts = split_sql_parts(sql)?;
 
-            let _ = cache.write().map(|mut cache| {
+            if let Ok(mut cache) = cache.write() {
                 cache.insert(sql.to_owned(), (parts.clone(), None));
-            });
+            } else {
+                tracing::warn!("Failed to acquire write lock for SQL statement cache");
+            }
 
             Ok(Self {
                 parts,