Always return an array of problems in report

puppetlabs · Feb 14, 2023 · 028ad1d · 028ad1d
1 parent 9fbfb66
commit 028ad1d
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/puppet-lint.rb b/lib/puppet-lint.rb
@@ -170,6 +170,8 @@ def print_context(message)
   # Returns array of problem.
   def report(problems)
     json = []
+    print_stdout = !(configuration.json || configuration.sarif)
+
     problems.each do |message|
       next if message[:kind] == :ignored && !PuppetLint.configuration.show_ignored
 
@@ -179,11 +181,9 @@ def report(problems)
 
       message[:context] = get_context(message) if configuration.with_context
 
-      if configuration.json || configuration.sarif || configuration.codeclimate_report_file
-        json << message
-      end
+      json << message
 
-      unless configuration.json || configuration.sarif
+      if print_stdout
         format_message(message)
         print_github_annotation(message) if configuration.github_actions
       end