feat: added ignore warnings to watch command

Author: raphael-goetz

crates/cli/src/analyser/core.rs

@@ -39,7 +39,7 @@ impl Analyser {
         super::loader::load_from_path(path)
     }
 
-    pub fn report(&mut self, will_exit: bool) {
+    pub fn report(&mut self, will_exit: bool, with_warning: bool) {
         // Run analysis passes
         for dt in self.data_types.clone() {
             self.analyse_data_type(&dt);
@@ -50,7 +50,7 @@ impl Analyser {
         for f in self.functions.clone() {
             self.analyse_runtime_function(&f);
         }
-        self.reporter.print(will_exit);
+        self.reporter.print(will_exit, true, with_warning);
     }
 
     pub fn data_type_identifier_exists(&self, identifier: &str, except_id: Option<i16>) -> bool {

crates/cli/src/command/feature.rs

@@ -14,7 +14,7 @@ pub fn search_feature(name: Option<String>, path: Option<String>) {
     };
 
     let mut analyser = Analyser::new(dir_path.as_str());
-    analyser.report(true);
+    analyser.report(true, true);
 
     let features = match name {
         None => parser.features.clone(),

crates/cli/src/command/push/mod.rs

@@ -20,7 +20,7 @@ pub async fn push(token: String, url: String, path: Option<String>) {
     info(String::from("Press Ctrl+C to stop watching..."));
 
     {
-        Analyser::new(dir_path.as_str()).report(false);
+        Analyser::new(dir_path.as_str()).report(false, true);
     }
 
     // Set up file watcher
@@ -82,7 +82,7 @@ pub async fn push(token: String, url: String, path: Option<String>) {
                                 .await;
                         }
 
-                        analyzer.report(false);
+                        analyzer.report(false, true);
 
                         last_run = Instant::now();
                     }
@@ -126,7 +126,7 @@ pub async fn push(token: String, url: String, path: Option<String>) {
                                 .await;
                         }
 
-                        analyzer.report(false);
+                        analyzer.report(false, true);
                         last_run = Instant::now();
                     }
                 }

crates/cli/src/command/report.rs

@@ -14,7 +14,7 @@ pub fn report_errors(path: Option<String>) {
     };
 
     let mut analyser = Analyser::new(dir_path.as_str());
-    analyser.report(true);
+    analyser.report(true, true);
 
     let rows = summary_table(&parser.features);
     success_table(rows);

crates/cli/src/command/watch.rs

@@ -5,14 +5,14 @@ use notify::{EventKind, RecursiveMode, Watcher, recommended_watcher};
 use std::sync::mpsc::channel;
 use std::time::{Duration, Instant};
 
-pub async fn watch_for_changes(path: Option<String>) {
+pub async fn watch_for_changes(path: Option<String>, with_warning: bool) {
     let dir_path = path.unwrap_or_else(|| "./definitions".to_string());
 
     info(format!("Watching directory: {dir_path}"));
     info(String::from("Press Ctrl+C to stop watching..."));
 
     {
-        Analyser::new(dir_path.as_str()).report(false);
+        Analyser::new(dir_path.as_str()).report(false, with_warning);
     }
 
     // Set up file watcher
@@ -35,7 +35,7 @@ pub async fn watch_for_changes(path: Option<String>) {
                             "\n\n\n--------------------------------------------------------------------------\n\n",
                         ));
                         info(String::from("Change detected! Regenerating report..."));
-                        Analyser::new(dir_path.as_str()).report(false);
+                        Analyser::new(dir_path.as_str()).report(false, with_warning);
                         last_run = Instant::now();
                     }
                 }
@@ -45,7 +45,7 @@ pub async fn watch_for_changes(path: Option<String>) {
                             "\n\n\n--------------------------------------------------------------------------\n\n",
                         ));
                         info(String::from("Change detected! Regenerating report..."));
-                        Analyser::new(dir_path.as_str()).report(false);
+                        Analyser::new(dir_path.as_str()).report(false, with_warning);
                         last_run = Instant::now();
                     }
                 }

crates/cli/src/diagnostics/reporter.rs

@@ -28,13 +28,19 @@ impl Reporter {
         self.diagnostics.is_empty()
     }
 
-    pub fn print(&self, will_exit: bool) {
-        for d in self.errors() {
-            println!("{}", d.print());
+    pub fn print(&self, will_exit: bool, print_errors: bool, print_warning: bool) {
+        if print_warning {
+            for d in self.warnings() {
+                println!("{}", d.print());
+            }
         }
-        for d in self.warnings() {
-            println!("{}", d.print());
+
+        if print_errors {
+            for d in self.errors() {
+                println!("{}", d.print());
+            }
         }
+
         if self
             .diagnostics
             .iter()

crates/cli/src/main.rs

@@ -49,6 +49,9 @@ enum Commands {
         /// Optional path to root directory of all definitions.
         #[arg(short, long)]
         path: Option<String>,
+        /// Should ignore warnings, true on default
+        #[arg(short, long, default_value_t = false)]
+        ignore_warnings: bool,
     },
     Push {
         /// Runtime Token for Sagittarius.
@@ -80,7 +83,7 @@ async fn main() {
         Commands::Download { tag, features } => {
             command::download::handle_download(tag, features).await
         }
-        Commands::Watch { path } => command::watch::watch_for_changes(path).await,
+        Commands::Watch { path, ignore_warnings } => command::watch::watch_for_changes(path, !ignore_warnings).await,
         Commands::Push { token, url, path } => command::push::push(token, url, path).await,
     }
 }