Clippy Fixes

- Addressed a Clippy warning by replacing `push_str("\n")` with `push('\n')` for better performance and code quality.
0xPlaygrounds · Sep 19, 2024 · 2babaa3 · 2babaa3
1 parent 5f05baa
commit 2babaa3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/rig-core/examples/rag_with_csv.rs b/rig-core/examples/rag_with_csv.rs
@@ -82,4 +82,4 @@ async fn main() -> Result<(), anyhow::Error> {
     println!("Agent Response:\n{}", response);
 
     Ok(())
-}
+}
diff --git a/rig-core/src/document_loaders/csv.rs b/rig-core/src/document_loaders/csv.rs
@@ -37,7 +37,7 @@ impl DocumentLoader for CsvLoader {
             for (i, field) in record.iter().enumerate() {
                 csv_content.push_str(&format!("{}: {}\n", headers[i], field));
             }
-            csv_content.push_str("\n");
+            csv_content.push('\n'); // Changed from push_str("\n") to push('\n')
         }
 
         Ok(vec![DocumentEmbeddings {
-Original file line number
+Diff line change
@@ Expand Up / @@ -82,4 +82,4 @@ async fn main() -> Result<(), anyhow::Error> { @@
         println!("Agent Response:\n{}", response);
         Ok(())
-    }
+    }