fix: cleanups; lint warnings fail CI (#9)

traceloop · Nov 15, 2024 · ee4f524 · ee4f524
1 parent 2d69d31
commit ee4f524
Show file tree

Hide file tree

Showing 8 changed files with 2 additions and 115 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -18,7 +18,7 @@ jobs:
       - name: Check formatting
         run: cargo fmt --check
       - name: Check clippy
-        run: cargo clippy --verbose
+        run: cargo clippy -- -D warnings
       - name: Build
         run: cargo build --verbose
       - name: Run tests

diff --git a/src/pipelines/mod.rs b/src/pipelines/mod.rs
@@ -1,4 +1 @@
 pub mod pipeline;
-pub mod plugin;
-pub mod plugins;
-pub mod services;
diff --git a/src/pipelines/pipeline.rs b/src/pipelines/pipeline.rs
@@ -35,7 +35,7 @@ pub fn create_pipeline(pipeline: &Pipeline, model_registry: &ModelRegistry) -> R
         };
     }
 
-    return router.with_state(Arc::new(model_registry.clone()));
+    router.with_state(Arc::new(model_registry.clone()))
 }
 
 pub async fn chat_completions(

diff --git a/src/pipelines/plugin.rs b/src/pipelines/plugin.rs
diff --git a/src/pipelines/plugins/logging.rs b/src/pipelines/plugins/logging.rs
diff --git a/src/pipelines/plugins/mod.rs b/src/pipelines/plugins/mod.rs
diff --git a/src/pipelines/plugins/tracing.rs b/src/pipelines/plugins/tracing.rs
diff --git a/src/pipelines/services/mod.rs b/src/pipelines/services/mod.rs
-Original file line number
+Diff line change
@@ Expand Up @@
             };
         }
-        return router.with_state(Arc::new(model_registry.clone()));
+        router.with_state(Arc::new(model_registry.clone()))
     }
     pub async fn chat_completions(
@@ Expand Down @@