move From<PolarsError> for CliError to clitypes.rs from joinp.rs

jqnatividad · Mar 12, 2024 · f58eaf3 · f58eaf3
1 parent d7721be
commit f58eaf3
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/src/clitypes.rs b/src/clitypes.rs
@@ -225,3 +225,10 @@ impl From<reqwest::Error> for CliError {
  CliError::Network(err.to_string())
  }
 }
+
+#[cfg(feature = "polars")]
+impl From<polars::error::PolarsError> for CliError {
+ fn from(err: polars::error::PolarsError) -> CliError {
+ CliError::Other(format!("Polars error: {err:?}"))
+ }
+}
diff --git a/src/cmd/joinp.rs b/src/cmd/joinp.rs
@@ -186,7 +186,7 @@ use polars::{
 use serde::Deserialize;
 use tempfile::tempdir;
 
-use crate::{cmd::sqlp::compress_output_if_needed, config::Delimiter, util, CliError, CliResult};
+use crate::{cmd::sqlp::compress_output_if_needed, config::Delimiter, util, CliResult};
 
 #[derive(Deserialize)]
 struct Args {
@@ -225,12 +225,6 @@ struct Args {
  flag_quiet: bool,
 }
 
-impl From<polars::error::PolarsError> for CliError {
- fn from(err: polars::error::PolarsError) -> CliError {
- CliError::Other(format!("Polars error: {err:?}"))
- }
-}
-
 pub fn run(argv: &[&str]) -> CliResult<()> {
  let mut args: Args = util::get_args(USAGE, argv)?;