Linting

lycheeverse · Jun 14, 2024 · ad70052 · ad70052
1 parent ebaf3ed
commit ad70052
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 23 deletions.
diff --git a/lychee-bin/src/commands/check.rs b/lychee-bin/src/commands/check.rs
@@ -340,10 +340,9 @@ fn get_failed_urls(stats: &mut ResponseStats) -> Vec<(InputSource, Url)> {
 
 #[cfg(test)]
 mod tests {
-    use crate::formatters;
     use crate::{formatters::get_response_formatter, options};
     use log::info;
-    use lychee_lib::{CacheStatus, ClientBuilder, InputSource, ResponseBody, Uri};
+    use lychee_lib::{CacheStatus, ClientBuilder, InputSource, Uri};
 
     use super::*;
 

diff --git a/lychee-bin/src/options.rs b/lychee-bin/src/options.rs
@@ -13,7 +13,7 @@ use secrecy::{ExposeSecret, SecretString};
 use serde::Deserialize;
 use std::path::Path;
 use std::{fs, path::PathBuf, str::FromStr, time::Duration};
-use strum::{Display, EnumIter, EnumString, EnumVariantNames, VariantNames};
+use strum::{Display, EnumIter, EnumString, VariantNames};
 
 pub(crate) const LYCHEE_IGNORE_FILE: &str = ".lycheeignore";
 pub(crate) const LYCHEE_CACHE_FILE: &str = ".lycheecache";
@@ -46,7 +46,7 @@ const TIMEOUT_STR: &str = concatcp!(DEFAULT_TIMEOUT_SECS);
 const RETRY_WAIT_TIME_STR: &str = concatcp!(DEFAULT_RETRY_WAIT_TIME_SECS);
 
 /// The format to use for the final status report
-#[derive(Debug, Deserialize, Default, Clone, Display, EnumIter, EnumVariantNames)]
+#[derive(Debug, Deserialize, Default, Clone, Display, EnumIter, VariantNames)]
 #[non_exhaustive]
 #[strum(serialize_all = "snake_case")]
 pub(crate) enum StatsFormat {
@@ -77,7 +77,7 @@ impl FromStr for StatsFormat {
 ///
 /// This decides over whether to use color,
 /// emojis, or plain text for the output.
-#[derive(Debug, Deserialize, Default, Clone, Display, EnumIter, EnumString, EnumVariantNames)]
+#[derive(Debug, Deserialize, Default, Clone, Display, EnumIter, EnumString, VariantNames)]
 #[non_exhaustive]
 pub(crate) enum OutputMode {
     #[serde(rename = "plain")]

diff --git a/lychee-lib/src/client.rs b/lychee-lib/src/client.rs
@@ -507,7 +507,7 @@ impl Client {
         let status = match uri.scheme() {
             _ if uri.is_file() => self.check_file(uri).await,
             _ if uri.is_mail() => self.check_mail(uri).await,
-            _ if uri.is_tel() => self.check_tel(uri).await,
+            _ if uri.is_tel() => Status::Excluded,
             _ => self.check_website(uri, default_chain).await?,
         };
 
@@ -700,6 +700,15 @@ impl Client {
         }
     }
 
+    /// Check a mail address, or equivalently a `mailto` URI.
+    ///
+    /// This implementation simply excludes all email addresses.
+    #[cfg(not(all(feature = "email-check", feature = "native-tls")))]
+    #[allow(clippy::unused_async)]
+    pub async fn check_mail(&self, _uri: &Uri) -> Status {
+        Status::Excluded
+    }
+
     /// Check a mail address, or equivalently a `mailto` URI.
     ///
     /// URIs may contain query parameters (e.g. `contact@example.com?subject="Hello"`),
@@ -717,23 +726,6 @@ impl Client {
             Status::Ok(StatusCode::OK)
         }
     }
-
-    /// Check a mail address, or equivalently a `mailto` URI.
-    ///
-    /// This implementation simply excludes all email addresses.
-    #[cfg(not(all(feature = "email-check", feature = "native-tls")))]
-    #[allow(clippy::unused_async)]
-    pub async fn check_mail(&self, _uri: &Uri) -> Status {
-        Status::Excluded
-    }
-
-    /// Check a tel
-    ///
-    /// This implementation simply excludes all tel.
-    #[allow(clippy::unused_async)]
-    pub async fn check_tel(&self, _uri: &Uri) -> Status {
-        Status::Excluded
-    }
 }
 
 // Check if the given `Url` would cause `reqwest` to panic.