Avoid puts to stdout

Author: martinemde

lib/query_packwerk/file_cache.rb

@@ -36,14 +36,14 @@ def initialize
     def load!(*file_names, headers: true)
       file_count = file_names.size
 
-      puts "Prepopulating AST cache with #{file_count} files: " if headers
+      warn "Prepopulating AST cache with #{file_count} files: " if headers
 
       file_names.each do |f|
         get_file_ast(f)
-        print '.'
+        $stderr.print '.'
       end
 
-      puts '', 'AST cache loaded' if headers
+      warn '', 'AST cache loaded' if headers
     end
 
     sig { params(cache_name: Symbol, key: T.untyped, value: T.untyped).returns(T.untyped) }

lib/query_packwerk/violations.rb

@@ -83,13 +83,13 @@ def initialize(original_collection, file_cache: QueryPackwerk::FileCache.new)
     def load_cache!
       return true if @cache_loaded
 
-      puts "Prepopulating AST cache with #{file_count} files: "
+      warn "Prepopulating AST cache with #{file_count} files: "
       start_time = Time.now
 
       @original_collection.each(&:load_cache!)
 
       finish_time = Time.now - start_time
-      puts '', "AST cache loaded in #{finish_time}"
+      warn '', "AST cache loaded in #{finish_time}"
       @cache_loaded = true
     end
 
@@ -99,19 +99,19 @@ def load_sources!
 
       unless @cache_loaded
         load_cache!
-        puts
+        warn
       end
 
-      puts "Prepopulating sources cache with #{count} violations: "
+      warn "Prepopulating sources cache with #{count} violations: "
       start_time = Time.now
 
       total_sources_loaded = @original_collection.sum do |violation|
-        print '.'
+        $stderr.print '.'
         violation.sources.size
       end
 
       finish_time = Time.now - start_time
-      puts "Loaded #{total_sources_loaded} full sources in #{finish_time}"
+      warn "Loaded #{total_sources_loaded} full sources in #{finish_time}"
 
       @sources_loaded = true
     end