diff --git a/crates/nu_plugin_polars/src/cache/mod.rs b/crates/nu_plugin_polars/src/cache/mod.rs
index 0fbf49f46f..49a76927e5 100644
--- a/crates/nu_plugin_polars/src/cache/mod.rs
+++ b/crates/nu_plugin_polars/src/cache/mod.rs
@@ -197,11 +197,11 @@ mod test {
 
     impl EngineWrapper for &MockEngineWrapper {
         fn get_env_var(&self, _key: &str) -> Option<String> {
-            todo!()
+            unimplemented!()
         }
 
         fn use_color(&self) -> bool {
-            todo!()
+            unimplemented!()
         }
 
         fn set_gc_disabled(&self, disabled: bool) -> Result<(), ShellError> {
diff --git a/crates/nu_plugin_polars/src/dataframe/values/utils.rs b/crates/nu_plugin_polars/src/dataframe/values/utils.rs
index 52704b82cf..5db17afbee 100644
--- a/crates/nu_plugin_polars/src/dataframe/values/utils.rs
+++ b/crates/nu_plugin_polars/src/dataframe/values/utils.rs
@@ -6,8 +6,6 @@ pub const DEFAULT_ROWS: usize = 5;
 
 // Converts a Vec<Value> to a Vec<Spanned<String>> with a Span marking the whole
 // location of the columns for error referencing
-// todo - fix
-#[allow(dead_code)]
 pub(crate) fn convert_columns(
     columns: Vec<Value>,
     span: Span,
@@ -49,8 +47,6 @@ pub(crate) fn convert_columns(
 
 // Converts a Vec<Value> to a Vec<String> with a Span marking the whole
 // location of the columns for error referencing
-// todo - fix
-#[allow(dead_code)]
 pub(crate) fn convert_columns_string(
     columns: Vec<Value>,
     span: Span,
@@ -92,8 +88,6 @@ pub(crate) fn convert_columns_string(
 
 // Converts a Vec<Value> to a Vec<String> with a Span marking the whole
 // location of the columns for error referencing
-// todo - fix
-#[allow(dead_code)]
 pub(crate) fn convert_columns_sm_str(
     columns: Vec<Value>,
     span: Span,