refactor: use Display trait instead of custom println

Author: fechu

src/cli.rs

@@ -3,7 +3,6 @@ use std::fmt::Display;
 use std::fs::File;
 use std::process::Command as ProcessCommand;
 
-use chrono_humanize::HumanTime;
 use clap::{Parser, Subcommand};
 
 use crate::common::NonEmptyString;
@@ -303,12 +302,7 @@ impl<T: FrameStore> CommandExecutor<T> {
         match self.frame_store.get_ongoing_frame() {
             None => Err(CliError::NoOngoingRecording),
             Some(frame) => {
-                println!(
-                    "Project {} started {}, ({})",
-                    frame.project(),
-                    HumanTime::from(*frame.start()),
-                    frame.start()
-                );
+                println!("{}", frame);
                 Ok(())
             }
         }

src/frame.rs

@@ -1,6 +1,10 @@
-use std::hash::{DefaultHasher, Hasher};
+use std::{
+    fmt::Display,
+    hash::{DefaultHasher, Hasher},
+};
 
 use chrono::{DateTime, Local, TimeZone};
+use chrono_humanize::HumanTime;
 
 use crate::{
     common::NonEmptyString,
@@ -107,6 +111,18 @@ impl Frame {
     }
 }
 
+impl Display for Frame {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        write!(
+            f,
+            "Project {} started {} ({})",
+            self.project,
+            HumanTime::from(self.start),
+            self.start
+        )
+    }
+}
+
 /// Represents a completed frame.
 /// A completed frame is guaranteed to have an end time.
 #[derive(Debug, Clone)]