refactor: remove Logger module

The module occurrences and usage was already replaced in
previous commits, so it's possible to remove the whole module.

Signed-off-by: Lorenzo Bertin Salvador <lorenzobs@usp.br>

Author: lorenzoberts

src/app/mod.rs

@@ -11,7 +11,7 @@ use tracing::{event, Level};
 use std::collections::{HashMap, HashSet};
 
 use crate::{
-    infrastructure::{logging::Logger, monitoring::logging::garbage_collector::collect_garbage},
+    infrastructure::monitoring::logging::garbage_collector::collect_garbage,
     log_on_error,
     lore::{
         lore_api_client::BlockingLoreAPIClient,
@@ -60,8 +60,7 @@ pub struct App {
 impl App {
     /// Creates a new instance of `App`. It dynamically loads configurations
     /// based on precedence (see [crate::app::Config::build]), app data
-    /// (available mailing lists, bookmarked patchsets, reviewed patchsets), and
-    /// initializes the Logger (see [crate::app::logging::Logger])
+    /// (available mailing lists, bookmarked patchsets, reviewed patchsets)
     ///
     /// # Returns
     ///
@@ -80,9 +79,6 @@ impl App {
 
         let lore_api_client = BlockingLoreAPIClient::default();
 
-        // Initialize the logger before the app starts
-        Logger::init_log_file(&config)?;
-
         event!(Level::INFO, "patch-hub started");
         collect_garbage(&config);
 

src/infrastructure/errors.rs

@@ -1,6 +1,6 @@
 use std::panic;
 
-use super::{logging::Logger, terminal::restore};
+use super::terminal::restore;
 
 /// This replaces the standard color_eyre panic and error hooks with hooks that
 /// restore the terminal before printing the panic or error.
@@ -11,7 +11,6 @@ pub fn install_hooks() -> color_eyre::Result<()> {
     let panic_hook = panic_hook.into_panic_hook();
     panic::set_hook(Box::new(move |panic_info| {
         restore().unwrap();
-        Logger::flush();
         panic_hook(panic_info);
     }));
 

src/infrastructure/logging/mod.rs

@@ -1,4 +1,3 @@
 pub mod errors;
-pub mod logging;
 pub mod monitoring;
 pub mod terminal;

src/infrastructure/mod.rs

@@ -12,7 +12,6 @@ use cli::Cli;
 use color_eyre::eyre::bail;
 use handler::run_app;
 use infrastructure::{
-    logging::Logger,
     monitoring::{init_monitoring, InitMonitoringProduct},
     terminal::{init, restore},
 };
@@ -62,7 +61,5 @@ fn main() -> color_eyre::Result<()> {
 
     event!(Level::INFO, "patch-hub finished");
 
-    Logger::flush();
-
     Ok(())
 }