Print proper counts from the reporter

nicorichard · Jun 12, 2024 · 17525da · 17525da
1 parent 0d1f9e1
commit 17525da
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/Sources/XCStringsLint/services/Reporter/CommandLineReporter.swift b/Sources/XCStringsLint/services/Reporter/CommandLineReporter.swift
@@ -26,16 +26,17 @@ struct CommandLineReporter: Reporter {
             }
         }
 
-        let errors = results.flatMap { result in
+        let validations = results.flatMap { result in
             result.validations
         }
-            .filter { $0.rule.severity == .error }
+
+        let errors = validations.filter { $0.rule.severity == .error }
 
         if !errors.isEmpty {
-            print("Found \(results.count) total xcstringlint issues, \(errors.count) serious")
+            print("Found \(results.count) total xcstringlint issues in \(results.count) keys, \(errors.count) serious")
             throw ExitCode.failure
         } else if !results.isEmpty {
-            print("Found \(results.count) total xcstringlint issues")
+            print("Found \(results.count) total xcstringlint issues in \(results.count) keys")
         }
     }
 }
diff --git a/Sources/XCStringsLint/services/Reporter/XcodeReporter.swift b/Sources/XCStringsLint/services/Reporter/XcodeReporter.swift
@@ -28,16 +28,17 @@ struct XcodeReporter: Reporter {
             }
         }
 
-        let errors = results.flatMap { result in
+        let validations = results.flatMap { result in
             result.validations
         }
-            .filter { $0.rule.severity == .error }
+
+        let errors = validations.filter { $0.rule.severity == .error }
 
         if !errors.isEmpty {
-            print(level: .error, "Found \(results.count) total xcstringlint issues, \(errors.count) serious")
+            print(level: .error, "Found \(validations.count) total xcstringlint issues in \(results.count) keys, \(errors.count) serious")
             throw ExitCode.failure
         } else if !results.isEmpty {
-            print(level: .warning, "Found \(results.count) total xcstringlint issues")
+            print(level: .warning, "Found \(validations.count) total xcstringlint issues in \(results.count) keys")
         }
     }
 }