From f996fd351b32032afefd065de476f0e0904f316b Mon Sep 17 00:00:00 2001
From: Awiteb <a@4rs.nl>
Date: Mon, 25 Aug 2025 13:22:40 +0000
Subject: [PATCH] chore: Move `repo::view::format_list` to `utils::format_iter`

Signed-off-by: Awiteb <a@4rs.nl>
---
 src/cli/commands/repo/view.rs | 26 +++++++-------------------
 src/nostr_utils/utils.rs      | 13 +++++++++++++
 2 files changed, 20 insertions(+), 19 deletions(-)

diff --git a/src/cli/commands/repo/view.rs b/src/cli/commands/repo/view.rs
index 886c004..3434b6f 100644
--- a/src/cli/commands/repo/view.rs
+++ b/src/cli/commands/repo/view.rs
@@ -14,8 +14,6 @@
 // You should have received a copy of the GNU General Public License
 // along with this program. If not, see <https://gnu.org/licenses/gpl-3.0.html>.
 
-use std::fmt;
-
 use clap::Args;
 use nostr::nips::nip19::ToBech32;
 
@@ -68,13 +66,16 @@ impl CommandRunner for ViewArgs {
                 repo_details.push_str(&format!("\nDescription: {desc}"));
             }
             if !repo.web.is_empty() {
-                repo_details.push_str(&format!("\nWebpages:\n{}", format_list(repo.web)));
+                repo_details.push_str(&format!("\nWebpages:\n{}", utils::format_iter(repo.web)));
             }
             if !repo.clone.is_empty() {
-                repo_details.push_str(&format!("\nClone urls:\n{}", format_list(repo.clone)));
+                repo_details.push_str(&format!(
+                    "\nClone urls:\n{}",
+                    utils::format_iter(repo.clone)
+                ));
             }
             if !repo.relays.is_empty() {
-                repo_details.push_str(&format!("\nRelays:\n{}", format_list(repo.relays)));
+                repo_details.push_str(&format!("\nRelays:\n{}", utils::format_iter(repo.relays)));
             }
             if let Some(euc) = repo.euc {
                 repo_details.push_str(&format!("\nEarliest unique commit: {euc}"));
@@ -82,7 +83,7 @@ impl CommandRunner for ViewArgs {
             if !repo.maintainers.is_empty() {
                 repo_details.push_str(&format!(
                     "\nMaintainers:\n{}",
-                    format_list(
+                    utils::format_iter(
                         repo.maintainers
                             .iter()
                             .map(|p| p.to_bech32().expect("Infallible"))
@@ -96,16 +97,3 @@ impl CommandRunner for ViewArgs {
         Ok(())
     }
 }
-
-/// Format a vector to print it
-fn format_list<I, T>(iterator: I) -> String
-where
-    I: IntoIterator<Item = T>,
-    T: fmt::Display,
-{
-    iterator
-        .into_iter()
-        .map(|t| format!(" - {t}"))
-        .collect::<Vec<String>>()
-        .join("\n")
-}
diff --git a/src/nostr_utils/utils.rs b/src/nostr_utils/utils.rs
index 701879e..bddbf74 100644
--- a/src/nostr_utils/utils.rs
+++ b/src/nostr_utils/utils.rs
@@ -338,3 +338,16 @@ pub fn subject_and_body(
         Ok((file_content.trim().to_owned(), None))
     }
 }
+
+/// Format an iterator to print it
+pub fn format_iter<I, T>(iterator: I) -> String
+where
+    I: IntoIterator<Item = T>,
+    T: fmt::Display,
+{
+    iterator
+        .into_iter()
+        .map(|t| format!(" - {t}"))
+        .collect::<Vec<String>>()
+        .join("\n")
+}