refacto: add info builder pattern (#1047)

* info builder pattern

* fix test

* reorganize methods

* fix bench

benches/repo.rs

@@ -1,6 +1,6 @@
 use criterion::{black_box, criterion_group, criterion_main, Criterion};
 use gix::{open, ThreadSafeRepository};
-use onefetch::{cli::CliOptions, info::Info};
+use onefetch::{cli::CliOptions, info::build_info};
 
 fn bench_repo_info(c: &mut Criterion) {
     let name = "repo.sh".to_string();
@@ -13,7 +13,7 @@ fn bench_repo_info(c: &mut Criterion) {
 
     c.bench_function("get repo information", |b| {
         b.iter(|| {
-            let result = black_box(Info::new(&config));
+            let result = black_box(build_info(&config));
             assert!(result.is_ok());
         })
     });

src/info/author.rs

@@ -1,10 +1,6 @@
 use crate::{
     cli::NumberSeparator,
-    info::{
-        format_number,
-        utils::git::Commits,
-        utils::info_field::{InfoField, InfoType},
-    },
+    info::{format_number, utils::git::Commits, utils::info_field::InfoField},
 };
 use owo_colors::{DynColors, OwoColorize};
 use serde::Serialize;
@@ -76,7 +72,7 @@ pub struct AuthorsInfo {
 }
 
 impl AuthorsInfo {
-    pub fn new(info_color: DynColors, commits: &mut Commits) -> Self {
+    pub fn new(info_color: DynColors, commits: &Commits) -> Self {
         let authors = commits.authors_to_display.clone();
         Self {
             authors,
@@ -119,10 +115,6 @@ impl InfoField for AuthorsInfo {
         title
     }
 
-    fn r#type(&self) -> InfoType {
-        InfoType::Authors
-    }
-
     fn should_color(&self) -> bool {
         false
     }

src/info/commits.rs

@@ -2,11 +2,7 @@ use serde::Serialize;
 
 use crate::{
     cli::NumberSeparator,
-    info::{
-        format_number,
-        utils::git::Commits,
-        utils::info_field::{InfoField, InfoType},
-    },
+    info::{format_number, utils::git::Commits, utils::info_field::InfoField},
 };
 
 #[derive(Serialize)]
@@ -41,10 +37,6 @@ impl InfoField for CommitsInfo {
     fn title(&self) -> String {
         "Commits".into()
     }
-
-    fn r#type(&self) -> InfoType {
-        InfoType::Commits
-    }
 }
 
 #[cfg(test)]

src/info/contributors.rs

@@ -2,11 +2,7 @@ use serde::Serialize;
 
 use crate::{
     cli::NumberSeparator,
-    info::{
-        format_number,
-        utils::git::Commits,
-        utils::info_field::{InfoField, InfoType},
-    },
+    info::{format_number, utils::git::Commits, utils::info_field::InfoField},
 };
 
 #[derive(Serialize)]
@@ -46,10 +42,6 @@ impl InfoField for ContributorsInfo {
     fn title(&self) -> String {
         "Contributors".into()
     }
-
-    fn r#type(&self) -> InfoType {
-        InfoType::Contributors
-    }
 }
 
 #[cfg(test)]

src/info/created.rs

@@ -1,8 +1,5 @@
 use super::utils::format_time;
-use crate::info::{
-    utils::git::Commits,
-    utils::info_field::{InfoField, InfoType},
-};
+use crate::info::{utils::git::Commits, utils::info_field::InfoField};
 use serde::Serialize;
 
 #[derive(Serialize)]
@@ -31,10 +28,6 @@ impl InfoField for CreatedInfo {
     fn title(&self) -> String {
         "Created".into()
     }
-
-    fn r#type(&self) -> InfoType {
-        InfoType::Created
-    }
 }
 
 #[cfg(test)]

src/info/dependencies.rs

@@ -1,9 +1,6 @@
 use crate::{
     cli::NumberSeparator,
-    info::{
-        format_number,
-        utils::info_field::{InfoField, InfoType},
-    },
+    info::{format_number, utils::info_field::InfoField},
 };
 use onefetch_manifest::Manifest;
 use serde::Serialize;
@@ -40,10 +37,6 @@ impl InfoField for DependenciesInfo {
     fn title(&self) -> String {
         "Dependencies".into()
     }
-
-    fn r#type(&self) -> InfoType {
-        InfoType::Dependencies
-    }
 }
 
 #[cfg(test)]

src/info/description.rs

@@ -1,4 +1,4 @@
-use crate::info::utils::info_field::{InfoField, InfoType};
+use crate::info::utils::info_field::InfoField;
 use onefetch_manifest::Manifest;
 use serde::Serialize;
 use std::fmt::Write;
@@ -50,10 +50,6 @@ impl InfoField for DescriptionInfo {
     fn title(&self) -> String {
         "Description".into()
     }
-
-    fn r#type(&self) -> InfoType {
-        InfoType::Description
-    }
 }
 
 #[cfg(test)]

src/info/head.rs

@@ -1,4 +1,4 @@
-use crate::info::utils::info_field::{InfoField, InfoType};
+use crate::info::utils::info_field::InfoField;
 use anyhow::{Context, Result};
 use gix::{reference::Category, Reference, Repository};
 use serde::Serialize;
@@ -72,10 +72,6 @@ impl InfoField for HeadInfo {
     fn title(&self) -> String {
         "HEAD".into()
     }
-
-    fn r#type(&self) -> InfoType {
-        InfoType::Head
-    }
 }
 
 #[cfg(test)]

src/info/langs/language.rs

@@ -1,4 +1,4 @@
-use crate::info::utils::info_field::{InfoField, InfoType};
+use crate::info::utils::info_field::InfoField;
 use owo_colors::OwoColorize;
 use serde::Serialize;
 use tokei;
@@ -158,9 +158,6 @@ impl InfoField for LanguagesInfo {
         title
     }
 
-    fn r#type(&self) -> InfoType {
-        InfoType::Languages
-    }
     fn should_color(&self) -> bool {
         false
     }

src/info/last_change.rs

@@ -1,8 +1,5 @@
 use super::utils::format_time;
-use crate::info::{
-    utils::git::Commits,
-    utils::info_field::{InfoField, InfoType},
-};
+use crate::info::{utils::git::Commits, utils::info_field::InfoField};
 use serde::Serialize;
 
 #[derive(Serialize)]
@@ -32,10 +29,6 @@ impl InfoField for LastChangeInfo {
     fn title(&self) -> String {
         "Last change".into()
     }
-
-    fn r#type(&self) -> InfoType {
-        InfoType::LastChange
-    }
 }
 
 #[cfg(test)]

src/info/license.rs

@@ -1,4 +1,4 @@
-use crate::info::utils::info_field::{InfoField, InfoType};
+use crate::info::utils::info_field::InfoField;
 use anyhow::{bail, Result};
 use askalono::{Store, TextData};
 use onefetch_manifest::Manifest;
@@ -95,10 +95,6 @@ impl InfoField for LicenseInfo {
     fn title(&self) -> String {
         "License".into()
     }
-
-    fn r#type(&self) -> InfoType {
-        InfoType::License
-    }
 }
 
 #[cfg(test)]

src/info/loc.rs

@@ -4,10 +4,7 @@ use serde::Serialize;
 
 use crate::{
     cli::NumberSeparator,
-    info::{
-        format_number,
-        utils::info_field::{InfoField, InfoType},
-    },
+    info::{format_number, utils::info_field::InfoField},
 };
 
 #[derive(Serialize)]
@@ -37,10 +34,6 @@ impl InfoField for LocInfo {
     fn title(&self) -> String {
         "Lines of code".into()
     }
-
-    fn r#type(&self) -> InfoType {
-        InfoType::LinesOfCode
-    }
 }
 
 #[cfg(test)]