Hide reporters from public interface

Source/SwiftLintCore/Models/Configuration.swift

@@ -127,7 +127,7 @@ public struct Configuration {
         excludedPaths: [String] = [],
         indentation: IndentationStyle = .default,
         warningThreshold: Int? = nil,
-        reporter: String = XcodeReporter.identifier,
+        reporter: String = "xcode",
         cachePath: String? = nil,
         pinnedVersion: String? = nil,
         allowZeroLintableFiles: Bool = false

Source/SwiftLintCore/Protocols/Reporter.swift

@@ -21,7 +21,7 @@ public protocol Reporter: CustomStringConvertible {
     static func generateReport(_ violations: [StyleViolation]) -> String
 }
 
-public extension Reporter {
+extension Reporter {
     /// For CustomStringConvertible conformance.
     var description: String { Self.description }
 }

Source/SwiftLintCore/Reporters/CSVReporter.swift

@@ -1,14 +1,14 @@
 import Foundation
 
 /// Reports violations as a newline-separated string of comma-separated values (CSV).
-public struct CSVReporter: Reporter {
+struct CSVReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "csv"
-    public static let isRealtime = false
-    public static let description = "Reports violations as a newline-separated string of comma-separated values (CSV)."
+    static let identifier = "csv"
+    static let isRealtime = false
+    static let description = "Reports violations as a newline-separated string of comma-separated values (CSV)."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         let keys = [
             "file",
             "line",

Source/SwiftLintCore/Reporters/CheckstyleReporter.swift

@@ -1,13 +1,13 @@
 /// Reports violations as XML conforming to the Checkstyle specification, as defined here:
 /// https://www.jetbrains.com/help/teamcity/xml-report-processing.html
-public struct CheckstyleReporter: Reporter {
+struct CheckstyleReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "checkstyle"
-    public static let isRealtime = false
-    public static let description = "Reports violations as Checkstyle XML."
+    static let identifier = "checkstyle"
+    static let isRealtime = false
+    static let description = "Reports violations as Checkstyle XML."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         return [
             "<?xml version=\"1.0\" encoding=\"utf-8\"?>\n<checkstyle version=\"4.3\">",
             violations

Source/SwiftLintCore/Reporters/CodeClimateReporter.swift

@@ -5,14 +5,14 @@ import Foundation
 import SourceKittenFramework
 
 /// Reports violations as a JSON array in Code Climate format.
-public struct CodeClimateReporter: Reporter {
+struct CodeClimateReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "codeclimate"
-    public static let isRealtime = false
-    public static let description = "Reports violations as a JSON array in Code Climate format."
+    static let identifier = "codeclimate"
+    static let isRealtime = false
+    static let description = "Reports violations as a JSON array in Code Climate format."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         return toJSON(violations.map(dictionary(for:)))
             .replacingOccurrences(of: "\\/", with: "/")
     }

Source/SwiftLintCore/Reporters/EmojiReporter.swift

@@ -1,12 +1,12 @@
 /// Reports violations in a format that's both fun and easy to read.
-public struct EmojiReporter: Reporter {
+struct EmojiReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "emoji"
-    public static let isRealtime = false
-    public static let description = "Reports violations in the format that's both fun and easy to read."
+    static let identifier = "emoji"
+    static let isRealtime = false
+    static let description = "Reports violations in the format that's both fun and easy to read."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         violations
             .group { $0.location.file ?? "Other" }
             .sorted { $0.key < $1.key }

Source/SwiftLintCore/Reporters/GitHubActionsLoggingReporter.swift

@@ -1,13 +1,13 @@
 /// Reports violations in the format GitHub-hosted virtual machine for Actions can recognize as messages.
-public struct GitHubActionsLoggingReporter: Reporter {
+struct GitHubActionsLoggingReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "github-actions-logging"
-    public static let isRealtime = true
-    public static let description = "Reports violations in the format GitHub-hosted virtual " +
+    static let identifier = "github-actions-logging"
+    static let isRealtime = true
+    static let description = "Reports violations in the format GitHub-hosted virtual " +
                                     "machine for Actions can recognize as messages."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         return violations.map(generateForSingleViolation).joined(separator: "\n")
     }
 

Source/SwiftLintCore/Reporters/GitLabJUnitReporter.swift

@@ -1,12 +1,12 @@
 /// Reports violations as JUnit XML supported by GitLab.
-public struct GitLabJUnitReporter: Reporter {
+struct GitLabJUnitReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "gitlab"
-    public static let isRealtime = false
-    public static let description = "Reports violations as JUnit XML supported by GitLab."
+    static let identifier = "gitlab"
+    static let isRealtime = false
+    static let description = "Reports violations as JUnit XML supported by GitLab."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         return "<?xml version=\"1.0\" encoding=\"utf-8\"?>\n<testsuites><testsuite>" +
             violations.map({ violation -> String in
                 let fileName = (violation.location.relativeFile ?? "<nopath>").escapedForXML()

Source/SwiftLintCore/Reporters/HTMLReporter.swift

@@ -7,14 +7,14 @@ private let formatter: DateFormatter = {
 }()
 
 /// Reports violations as HTML.
-public struct HTMLReporter: Reporter {
+struct HTMLReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "html"
-    public static let isRealtime = false
-    public static let description = "Reports violations as HTML."
+    static let identifier = "html"
+    static let isRealtime = false
+    static let description = "Reports violations as HTML."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         return generateReport(violations, swiftlintVersion: Version.current.value,
                               dateString: formatter.string(from: Date()))
     }

Source/SwiftLintCore/Reporters/JSONReporter.swift

@@ -2,14 +2,14 @@ import Foundation
 import SourceKittenFramework
 
 /// Reports violations as a JSON array.
-public struct JSONReporter: Reporter {
+struct JSONReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "json"
-    public static let isRealtime = false
-    public static let description: String = "Reports violations as a JSON array."
+    static let identifier = "json"
+    static let isRealtime = false
+    static let description: String = "Reports violations as a JSON array."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         return toJSON(violations.map(dictionary(for:)))
     }
 

Source/SwiftLintCore/Reporters/JUnitReporter.swift

@@ -1,12 +1,12 @@
 /// Reports violations as JUnit XML.
-public struct JUnitReporter: Reporter {
+struct JUnitReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "junit"
-    public static let isRealtime = false
-    public static let description = "Reports violations as JUnit XML."
+    static let identifier = "junit"
+    static let isRealtime = false
+    static let description = "Reports violations as JUnit XML."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         let warningCount = violations.filter({ $0.severity == .warning }).count
         let errorCount = violations.filter({ $0.severity == .error }).count
 

Source/SwiftLintCore/Reporters/MarkdownReporter.swift

@@ -1,14 +1,14 @@
 import Foundation
 
 /// Reports violations as markdown formated (with tables).
-public struct MarkdownReporter: Reporter {
+struct MarkdownReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "markdown"
-    public static let isRealtime = false
-    public static let description = "Reports violations as markdown formated (with tables)."
+    static let identifier = "markdown"
+    static let isRealtime = false
+    static let description = "Reports violations as markdown formated (with tables)."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         let keys = [
             "file",
             "line",

Source/SwiftLintCore/Reporters/RelativePathReporter.swift

@@ -1,12 +1,12 @@
 /// Reports violations with relative paths.
-public struct RelativePathReporter: Reporter {
+struct RelativePathReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "relative-path"
-    public static let isRealtime = true
-    public static let description = "Reports violations with relative paths."
+    static let identifier = "relative-path"
+    static let isRealtime = true
+    static let description = "Reports violations with relative paths."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         return violations.map(generateForSingleViolation).joined(separator: "\n")
     }
 

Source/SwiftLintCore/Reporters/SonarQubeReporter.swift

@@ -1,14 +1,14 @@
 import SourceKittenFramework
 
 /// Reports violations in SonarQube import format.
-public struct SonarQubeReporter: Reporter {
+struct SonarQubeReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "sonarqube"
-    public static let isRealtime = false
-    public static let description = "Reports violations in SonarQube import format."
+    static let identifier = "sonarqube"
+    static let isRealtime = false
+    static let description = "Reports violations in SonarQube import format."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         return toJSON(["issues": violations.map(dictionary(for:))])
     }
 

Source/SwiftLintCore/Reporters/SummaryReporter.swift

@@ -2,15 +2,15 @@ import Foundation
 import SwiftyTextTable
 
 /// Reports a summary table of all violations
-public struct SummaryReporter: Reporter {
+struct SummaryReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "summary"
-    public static let isRealtime = false
+    static let identifier = "summary"
+    static let isRealtime = false
 
-    public static let description = "Reports a summary table of all violations."
+    static let description = "Reports a summary table of all violations."
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         TextTable(violations: violations).renderWithExtraSeparator()
     }
 }

Source/SwiftLintCore/Reporters/XcodeReporter.swift

@@ -1,12 +1,12 @@
 /// Reports violations in the format Xcode uses to display in the IDE. (default)
-public struct XcodeReporter: Reporter {
+struct XcodeReporter: Reporter {
     // MARK: - Reporter Conformance
 
-    public static let identifier = "xcode"
-    public static let isRealtime = true
-    public static let description = "Reports violations in the format Xcode uses to display in the IDE. (default)"
+    static let identifier = "xcode"
+    static let isRealtime = true
+    static let description = "Reports violations in the format Xcode uses to display in the IDE. (default)"
 
-    public static func generateReport(_ violations: [StyleViolation]) -> String {
+    static func generateReport(_ violations: [StyleViolation]) -> String {
         return violations.map(generateForSingleViolation).joined(separator: "\n")
     }