Merge pull request #77 from shadr/fix-logging

Print logs to the stderr instead of stdout

Author: NathanLovato

src/main.rs

@@ -112,7 +112,7 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
                 eprintln!("The input passed via stdin is not formatted");
                 std::process::exit(1);
             } else {
-                println!("The input passed via stdin is already formatted");
+                eprintln!("The input passed via stdin is already formatted");
             }
         } else {
             print!("{}", formatted_content);
@@ -136,7 +136,7 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
 
     let total_files = input_gdscript_files.len();
 
-    print!(
+    eprint!(
         "Formatting {} file{}...",
         total_files,
         if total_files == 1 { "" } else { "s" }
@@ -192,6 +192,9 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
                 } else if args.stdout {
                     // Clear the progress message before printing formatted files to stdout
                     terminal_clear_line();
+                    // A little bit hacky, but because terminals by default output both stdout and stderr
+                    // we need to return carriage to the start to print formatted output from the start of the line
+                    eprint!("\r");
                     // If there are multiple input files we still allow stdout but we print a separator
                     if total_files > 1 {
                         println!("#--file:{}", output.file_path.display());
@@ -216,15 +219,15 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
     if args.check {
         if all_formatted {
             terminal_clear_line();
-            println!("\rAll {} file(s) are formatted", total_files);
+            eprintln!("\rAll {} file(s) are formatted", total_files);
         } else {
             terminal_clear_line();
             eprintln!("\rSome files are not formatted");
             std::process::exit(1);
         }
     } else if !args.stdout {
         terminal_clear_line();
-        println!(
+        eprintln!(
             "\rFormatted {} file{}",
             total_files,
             if total_files == 1 { "" } else { "s" }
@@ -235,5 +238,5 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
 }
 
 fn terminal_clear_line() {
-    print!("\r{}", " ".repeat(80));
+    eprint!("\r{}", " ".repeat(80));
 }