chore(clippy): fix clippy warnings

Author: nixxoq

src/api/versions.rs

@@ -41,17 +41,14 @@ impl ModrinthAPI {
 
         let mut url = BASE_URL.join_all(vec!["project", project_id, "version"]);
 
-        match extra_options {
-            Some(extra_options) => {
-                url = BASE_URL.join_all(vec![
-                    "project", project_id, "version",
-                    // extra_options.number.unwrap_or(""),
-                ]);
-                url = url.add_optional_query_json("loaders", extra_options.loaders)?;
-                url = url.add_optional_query_json("game_versions", extra_options.game_versions)?;
-                url = url.add_optional_query_json("featured", extra_options.featured)?;
-            }
-            None => {}
+        if let Some(extra_options) = extra_options {
+            url = BASE_URL.join_all(vec![
+                "project", project_id, "version",
+                // extra_options.number.unwrap_or(""),
+            ]);
+            url = url.add_optional_query_json("loaders", extra_options.loaders)?;
+            url = url.add_optional_query_json("game_versions", extra_options.game_versions)?;
+            url = url.add_optional_query_json("featured", extra_options.featured)?;
         }
 
         self.client.get(url).custom_send_json().await
@@ -110,19 +107,16 @@ impl ModrinthAPI {
 
         let mut url = BASE_URL.join_all(vec!["project", project_id, "version"]);
 
-        match extra_options {
-            Some(extra_options) => {
-                url = BASE_URL.join_all(vec![
-                    "project",
-                    project_id,
-                    "version",
-                    extra_options.number.unwrap(),
-                ]);
-                url = url.add_optional_query_json("loaders", extra_options.loaders)?;
-                url = url.add_optional_query_json("game_versions", extra_options.game_versions)?;
-                url = url.add_optional_query_json("featured", extra_options.featured)?;
-            }
-            None => {}
+        if let Some(extra_options) = extra_options {
+            url = BASE_URL.join_all(vec![
+                "project",
+                project_id,
+                "version",
+                extra_options.number.unwrap(),
+            ]);
+            url = url.add_optional_query_json("loaders", extra_options.loaders)?;
+            url = url.add_optional_query_json("game_versions", extra_options.game_versions)?;
+            url = url.add_optional_query_json("featured", extra_options.featured)?;
         }
 
         self.client.get(url).custom_send_json().await

src/structs/versions.rs

@@ -40,22 +40,13 @@ pub struct Version {
 ///     If `Some(false)`, featured versions will be excluded. If `None`, both featured and
 ///     non-featured versions are included. This is only applied if `number` is `None`.
 ///     Example: `featured: Some(true)`
-#[derive(Debug)]
+#[derive(Debug, Default)]
 pub struct ProjectVersionsFilter<'a> {
     pub loaders: Option<&'a [&'a str]>,
     pub game_versions: Option<&'a [&'a str]>,
     pub featured: Option<bool>,
 }
 
-impl<'a> Default for ProjectVersionsFilter<'a> {
-    fn default() -> Self {
-        ProjectVersionsFilter {
-            loaders: None,
-            game_versions: None,
-            featured: None,
-        }
-    }
-}
 
 /// Extra parameters for [ModrinthAPI::get_project_version] function
 ///