From cd6acff635919b40f31553437306be2d8ceda550 Mon Sep 17 00:00:00 2001 From: Kyle Kelley Date: Fri, 19 Apr 2024 18:59:12 -0700 Subject: [PATCH] Remove TODOs from semantic index build-up (#10789) Quick touch up of some `// todo!()` that got left in `semantic-index` Release Notes: - N/A --- crates/semantic_index/src/semantic_index.rs | 2 -- 1 file changed, 2 deletions(-) diff --git a/crates/semantic_index/src/semantic_index.rs b/crates/semantic_index/src/semantic_index.rs index 86c18c6fe6..a43d9e177c 100644 --- a/crates/semantic_index/src/semantic_index.rs +++ b/crates/semantic_index/src/semantic_index.rs @@ -599,7 +599,6 @@ impl WorktreeIndex { let mut embeddings = Vec::new(); for embedding_batch in chunks.chunks(embedding_provider.batch_size()) { - // todo!("add a retry facility") embeddings.extend(embedding_provider.embed(embedding_batch).await?); } @@ -889,7 +888,6 @@ mod tests { .await .unwrap(); - // todo!(): use a fixture let project_path = Path::new("./fixture"); let project = cx