From 0fbd03cf2be37bd75b155a4bd3effa9e45659862 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Danny=20M=C3=B6sch?= Date: Sun, 16 Oct 2022 18:40:12 +0200 Subject: [PATCH] Require visitors to exist (#4383) --- Source/SwiftLintFramework/Protocols/SwiftSyntaxRule.swift | 7 +++---- .../AnonymousArgumentInMultilineClosureRule.swift | 2 +- .../Rules/Idiomatic/BlockBasedKVORule.swift | 2 +- .../Rules/Idiomatic/DiscouragedAssertRule.swift | 2 +- .../Rules/Idiomatic/DiscouragedObjectLiteralRule.swift | 2 +- .../Rules/Idiomatic/DiscouragedOptionalBooleanRule.swift | 2 +- .../Rules/Idiomatic/ExplicitEnumRawValueRule.swift | 2 +- .../Rules/Idiomatic/ExplicitInitRule.swift | 2 +- .../Rules/Idiomatic/FallthroughRule.swift | 2 +- .../SwiftLintFramework/Rules/Idiomatic/ForceCastRule.swift | 2 +- .../SwiftLintFramework/Rules/Idiomatic/ForceTryRule.swift | 2 +- .../Rules/Idiomatic/ForceUnwrappingRule.swift | 2 +- .../Idiomatic/FunctionDefaultParameterAtEndRule.swift | 2 +- .../Rules/Idiomatic/IsDisjointRule.swift | 2 +- .../Rules/Idiomatic/LegacyCGGeometryFunctionsRule.swift | 2 +- .../Rules/Idiomatic/LegacyConstantRule.swift | 2 +- .../Rules/Idiomatic/LegacyHashingRule.swift | 2 +- .../Rules/Idiomatic/LegacyMultipleRule.swift | 2 +- .../Rules/Idiomatic/LegacyNSGeometryFunctionsRule.swift | 2 +- .../Rules/Idiomatic/LegacyRandomRule.swift | 2 +- .../Rules/Idiomatic/NoExtensionAccessModifierRule.swift | 2 +- .../Rules/Idiomatic/NoFallthroughOnlyRule.swift | 2 +- .../Rules/Idiomatic/PrivateOverFilePrivateRule.swift | 2 +- .../Rules/Idiomatic/RedundantNilCoalescingRule.swift | 2 +- .../Rules/Idiomatic/RedundantStringEnumValueRule.swift | 2 +- .../Rules/Idiomatic/ReturnValueFromVoidFunctionRule.swift | 2 +- .../Rules/Idiomatic/ShorthandOptionalBindingRule.swift | 2 +- .../Rules/Idiomatic/StaticOperatorRule.swift | 2 +- .../Rules/Idiomatic/StrictFilePrivateRule.swift | 2 +- .../Rules/Idiomatic/ToggleBoolRule.swift | 2 +- .../Rules/Idiomatic/TrailingSemicolonRule.swift | 2 +- .../Rules/Idiomatic/UnavailableConditionRule.swift | 2 +- .../Rules/Idiomatic/UnneededBreakInSwitchRule.swift | 2 +- .../Rules/Idiomatic/UntypedErrorInCatchRule.swift | 2 +- .../Idiomatic/VoidFunctionInTernaryConditionRule.swift | 2 +- .../Rules/Idiomatic/XCTFailMessageRule.swift | 2 +- .../Rules/Lint/AnyObjectProtocolRule.swift | 2 +- Source/SwiftLintFramework/Rules/Lint/ArrayInitRule.swift | 2 +- .../Rules/Lint/ClassDelegateProtocolRule.swift | 2 +- .../Rules/Lint/DeploymentTargetRule.swift | 2 +- .../Rules/Lint/DuplicateEnumCasesRule.swift | 2 +- .../SwiftLintFramework/Rules/Lint/DynamicInlineRule.swift | 2 +- .../Rules/Lint/EmptyXCTestMethodRule.swift | 2 +- .../Rules/Lint/IBInspectableInExtensionRule.swift | 2 +- .../Rules/Lint/IdenticalOperandsRule.swift | 2 +- Source/SwiftLintFramework/Rules/Lint/InertDeferRule.swift | 2 +- .../Rules/Lint/NSLocalizedStringRequireBundleRule.swift | 2 +- .../Rules/Lint/NSObjectPreferIsEqualRule.swift | 2 +- .../SwiftLintFramework/Rules/Lint/PrivateActionRule.swift | 2 +- .../SwiftLintFramework/Rules/Lint/PrivateOutletRule.swift | 2 +- .../Rules/Lint/PrivateUnitTestRule.swift | 2 +- .../Rules/Lint/ProhibitedInterfaceBuilderRule.swift | 2 +- .../Rules/Lint/QuickDiscouragedFocusedTestRule.swift | 2 +- .../Rules/Lint/QuickDiscouragedPendingTestRule.swift | 2 +- .../Rules/Lint/SelfInPropertyInitializationRule.swift | 2 +- .../SwiftLintFramework/Rules/Lint/StrongIBOutletRule.swift | 2 +- .../Rules/Lint/UnownedVariableCaptureRule.swift | 2 +- .../Rules/Lint/UnusedClosureParameterRule.swift | 2 +- .../Rules/Lint/UnusedSetterValueRule.swift | 2 +- .../Rules/Lint/ValidIBInspectableRule.swift | 2 +- .../SwiftLintFramework/Rules/Lint/YodaConditionRule.swift | 2 +- .../Rules/Performance/ContainsOverFilterCountRule.swift | 2 +- .../Rules/Performance/ContainsOverFilterIsEmptyRule.swift | 2 +- .../Performance/ContainsOverRangeNilComparisonRule.swift | 2 +- .../Rules/Performance/EmptyCollectionLiteralRule.swift | 2 +- .../Rules/Performance/EmptyStringRule.swift | 2 +- .../Rules/Performance/FirstWhereRule.swift | 2 +- .../Rules/Performance/FlatMapOverMapReduceRule.swift | 2 +- .../Rules/Performance/LastWhereRule.swift | 2 +- .../Rules/Performance/ReduceBooleanRule.swift | 2 +- .../Rules/Performance/SortedFirstLastRule.swift | 2 +- Source/SwiftLintFramework/Rules/Style/AttributesRule.swift | 2 +- .../SwiftLintFramework/Rules/Style/ClosingBraceRule.swift | 2 +- .../Rules/Style/ClosureParameterPositionRule.swift | 2 +- .../Rules/Style/ClosureSpacingRule.swift | 2 +- .../Rules/Style/ComputedAccessorsOrderRule.swift | 2 +- .../Rules/Style/ConditionalReturnsOnNewlineRule.swift | 2 +- .../Rules/Style/EmptyEnumArgumentsRule.swift | 2 +- .../Rules/Style/EmptyParametersRule.swift | 2 +- .../Style/EmptyParenthesesWithTrailingClosureRule.swift | 2 +- .../Rules/Style/ImplicitGetterRule.swift | 2 +- .../Style/MultipleClosuresWithTrailingClosureRule.swift | 2 +- .../Rules/Style/NoSpaceInMethodCallRule.swift | 2 +- .../Rules/Style/NumberSeparatorRule.swift | 2 +- .../Rules/Style/OperatorFunctionWhitespaceRule.swift | 2 +- .../Rules/Style/ProtocolPropertyAccessorsOrderRule.swift | 2 +- .../Rules/Style/RedundantDiscardableLetRule.swift | 2 +- .../SwiftLintFramework/Rules/Style/SelfBindingRule.swift | 2 +- .../Rules/Style/ShorthandOperatorRule.swift | 2 +- .../Rules/Style/SwitchCaseOnNewlineRule.swift | 2 +- .../Style/UnneededParenthesesInClosureArgumentRule.swift | 2 +- 91 files changed, 93 insertions(+), 94 deletions(-) diff --git a/Source/SwiftLintFramework/Protocols/SwiftSyntaxRule.swift b/Source/SwiftLintFramework/Protocols/SwiftSyntaxRule.swift index ebfc7db9e..7956bf285 100644 --- a/Source/SwiftLintFramework/Protocols/SwiftSyntaxRule.swift +++ b/Source/SwiftLintFramework/Protocols/SwiftSyntaxRule.swift @@ -7,7 +7,7 @@ public protocol SwiftSyntaxRule: SourceKitFreeRule { /// - parameter file: The file for which to produce the visitor. /// /// - returns: A `ViolationsSyntaxVisitor` for the given file. - func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? + func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor /// Produce a violation for the given file and absolute position. /// @@ -53,12 +53,11 @@ public extension SwiftSyntaxRule { } func validate(file: SwiftLintFile) -> [StyleViolation] { - guard let visitor = makeVisitor(file: file), - let syntaxTree = preprocess(syntaxTree: file.syntaxTree) else { + guard let syntaxTree = preprocess(syntaxTree: file.syntaxTree) else { return [] } - return visitor + return makeVisitor(file: file) .walk(tree: syntaxTree, handler: \.violations) .sorted() .map { makeViolation(file: file, violation: $0) } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/AnonymousArgumentInMultilineClosureRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/AnonymousArgumentInMultilineClosureRule.swift index 9b062ca2d..bf6e48d4b 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/AnonymousArgumentInMultilineClosureRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/AnonymousArgumentInMultilineClosureRule.swift @@ -33,7 +33,7 @@ public struct AnonymousArgumentInMultilineClosureRule: SwiftSyntaxRule, OptInRul ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(locationConverter: file.locationConverter) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/BlockBasedKVORule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/BlockBasedKVORule.swift index e11ba540d..86d52d282 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/BlockBasedKVORule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/BlockBasedKVORule.swift @@ -35,7 +35,7 @@ public struct BlockBasedKVORule: SwiftSyntaxRule, ConfigurationProviderRule { ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedAssertRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedAssertRule.swift index 3f9be4ed0..507185c1a 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedAssertRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedAssertRule.swift @@ -31,7 +31,7 @@ public struct DiscouragedAssertRule: SwiftSyntaxRule, OptInRule, ConfigurationPr // MARK: - Public - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedObjectLiteralRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedObjectLiteralRule.swift index 4ae683398..46295d353 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedObjectLiteralRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedObjectLiteralRule.swift @@ -24,7 +24,7 @@ public struct DiscouragedObjectLiteralRule: SwiftSyntaxRule, OptInRule, Configur ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(configuration: configuration) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedOptionalBooleanRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedOptionalBooleanRule.swift index df0296396..4aac9fa13 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedOptionalBooleanRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/DiscouragedOptionalBooleanRule.swift @@ -14,7 +14,7 @@ public struct DiscouragedOptionalBooleanRule: OptInRule, ConfigurationProviderRu triggeringExamples: DiscouragedOptionalBooleanRuleExamples.triggeringExamples ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/ExplicitEnumRawValueRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/ExplicitEnumRawValueRule.swift index a66065197..997545a94 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/ExplicitEnumRawValueRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/ExplicitEnumRawValueRule.swift @@ -79,7 +79,7 @@ public struct ExplicitEnumRawValueRule: SwiftSyntaxRule, OptInRule, Configuratio ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/ExplicitInitRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/ExplicitInitRule.swift index 048bf02b3..6d2addf2c 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/ExplicitInitRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/ExplicitInitRule.swift @@ -171,7 +171,7 @@ public struct ExplicitInitRule: SwiftSyntaxCorrectableRule, ConfigurationProvide ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/FallthroughRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/FallthroughRule.swift index b8e7f5ac0..7c7d539c5 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/FallthroughRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/FallthroughRule.swift @@ -30,7 +30,7 @@ public struct FallthroughRule: SwiftSyntaxRule, ConfigurationProviderRule, OptIn ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/ForceCastRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/ForceCastRule.swift index e1850ecf2..f9e253782 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/ForceCastRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/ForceCastRule.swift @@ -16,7 +16,7 @@ public struct ForceCastRule: ConfigurationProviderRule, SwiftSyntaxRule { triggeringExamples: [ Example("NSNumber() ↓as! Int\n") ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { ForceCastRuleVisitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/ForceTryRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/ForceTryRule.swift index 461fe95fe..fcf5e70a4 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/ForceTryRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/ForceTryRule.swift @@ -26,7 +26,7 @@ public struct ForceTryRule: ConfigurationProviderRule, SwiftSyntaxRule { ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/ForceUnwrappingRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/ForceUnwrappingRule.swift index edb09d826..e6f41f3c3 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/ForceUnwrappingRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/ForceUnwrappingRule.swift @@ -67,7 +67,7 @@ public struct ForceUnwrappingRule: OptInRule, SwiftSyntaxRule, ConfigurationProv ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { ForceUnwrappingVisitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/FunctionDefaultParameterAtEndRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/FunctionDefaultParameterAtEndRule.swift index b96dbb97c..7e4e6abeb 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/FunctionDefaultParameterAtEndRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/FunctionDefaultParameterAtEndRule.swift @@ -51,7 +51,7 @@ public struct FunctionDefaultParameterAtEndRule: SwiftSyntaxRule, ConfigurationP ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/IsDisjointRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/IsDisjointRule.swift index 16551cef9..dc065d3d7 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/IsDisjointRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/IsDisjointRule.swift @@ -22,7 +22,7 @@ public struct IsDisjointRule: SwiftSyntaxRule, ConfigurationProviderRule { ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyCGGeometryFunctionsRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyCGGeometryFunctionsRule.swift index 67ccfa0f7..84482f3ae 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyCGGeometryFunctionsRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyCGGeometryFunctionsRule.swift @@ -102,7 +102,7 @@ public struct LegacyCGGeometryFunctionsRule: SwiftSyntaxCorrectableRule, Configu "CGRectIntersection": .function(name: "intersect", argumentLabels: [""]) ] - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { LegacyFunctionRuleHelper.Visitor(legacyFunctions: Self.legacyFunctions) } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyConstantRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyConstantRule.swift index 8f48a1871..f08c614f1 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyConstantRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyConstantRule.swift @@ -16,7 +16,7 @@ public struct LegacyConstantRule: SwiftSyntaxCorrectableRule, ConfigurationProvi corrections: LegacyConstantRuleExamples.corrections ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyHashingRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyHashingRule.swift index c10479c8d..8ff47b8fa 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyHashingRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyHashingRule.swift @@ -75,7 +75,7 @@ public struct LegacyHashingRule: SwiftSyntaxRule, ConfigurationProviderRule { ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyMultipleRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyMultipleRule.swift index 8c69f6da0..5d81560a7 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyMultipleRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyMultipleRule.swift @@ -43,7 +43,7 @@ public struct LegacyMultipleRule: OptInRule, ConfigurationProviderRule, SwiftSyn syntaxTree.folded() } - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyNSGeometryFunctionsRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyNSGeometryFunctionsRule.swift index dc5782101..517544121 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyNSGeometryFunctionsRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyNSGeometryFunctionsRule.swift @@ -102,7 +102,7 @@ public struct LegacyNSGeometryFunctionsRule: SwiftSyntaxCorrectableRule, Configu "NSPointInRect": .function(name: "contains", argumentLabels: [""], reversed: true) ] - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { LegacyFunctionRuleHelper.Visitor(legacyFunctions: Self.legacyFunctions) } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyRandomRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyRandomRule.swift index ceab7f4c3..fb2b56a29 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/LegacyRandomRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/LegacyRandomRule.swift @@ -22,7 +22,7 @@ public struct LegacyRandomRule: SwiftSyntaxRule, ConfigurationProviderRule { ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/NoExtensionAccessModifierRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/NoExtensionAccessModifierRule.swift index 84b91ff0e..46a1af126 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/NoExtensionAccessModifierRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/NoExtensionAccessModifierRule.swift @@ -23,7 +23,7 @@ public struct NoExtensionAccessModifierRule: SwiftSyntaxRule, OptInRule, Configu ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/NoFallthroughOnlyRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/NoFallthroughOnlyRule.swift index 3e5afbe98..7d379e201 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/NoFallthroughOnlyRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/NoFallthroughOnlyRule.swift @@ -14,7 +14,7 @@ public struct NoFallthroughOnlyRule: SwiftSyntaxRule, ConfigurationProviderRule triggeringExamples: NoFallthroughOnlyRuleExamples.triggeringExamples ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/PrivateOverFilePrivateRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/PrivateOverFilePrivateRule.swift index b28ae6820..df5433272 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/PrivateOverFilePrivateRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/PrivateOverFilePrivateRule.swift @@ -56,7 +56,7 @@ public struct PrivateOverFilePrivateRule: ConfigurationProviderRule, SwiftSyntax ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(validateExtensions: configuration.validateExtensions) } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/RedundantNilCoalescingRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/RedundantNilCoalescingRule.swift index bef76e14d..291d06967 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/RedundantNilCoalescingRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/RedundantNilCoalescingRule.swift @@ -23,7 +23,7 @@ public struct RedundantNilCoalescingRule: OptInRule, SwiftSyntaxCorrectableRule, ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/RedundantStringEnumValueRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/RedundantStringEnumValueRule.swift index 32e9f9544..165547087 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/RedundantStringEnumValueRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/RedundantStringEnumValueRule.swift @@ -61,7 +61,7 @@ public struct RedundantStringEnumValueRule: SwiftSyntaxRule, ConfigurationProvid ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/ReturnValueFromVoidFunctionRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/ReturnValueFromVoidFunctionRule.swift index ffca58ed5..6d9603465 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/ReturnValueFromVoidFunctionRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/ReturnValueFromVoidFunctionRule.swift @@ -15,7 +15,7 @@ public struct ReturnValueFromVoidFunctionRule: ConfigurationProviderRule, OptInR triggeringExamples: ReturnValueFromVoidFunctionRuleExamples.triggeringExamples ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { ReturnValueFromVoidFunctionVisitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/ShorthandOptionalBindingRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/ShorthandOptionalBindingRule.swift index 192dc083c..b6b3674a9 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/ShorthandOptionalBindingRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/ShorthandOptionalBindingRule.swift @@ -81,7 +81,7 @@ public struct ShorthandOptionalBindingRule: OptInRule, SwiftSyntaxCorrectableRul public init() {} - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/StaticOperatorRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/StaticOperatorRule.swift index fdb3ce5ef..2e1c66be4 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/StaticOperatorRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/StaticOperatorRule.swift @@ -78,7 +78,7 @@ public struct StaticOperatorRule: SwiftSyntaxRule, ConfigurationProviderRule, Op ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/StrictFilePrivateRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/StrictFilePrivateRule.swift index d7a237e54..bd0f0fdff 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/StrictFilePrivateRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/StrictFilePrivateRule.swift @@ -58,7 +58,7 @@ public struct StrictFilePrivateRule: OptInRule, ConfigurationProviderRule, Swift ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/ToggleBoolRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/ToggleBoolRule.swift index dee85eda1..c80a9614e 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/ToggleBoolRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/ToggleBoolRule.swift @@ -31,7 +31,7 @@ public struct ToggleBoolRule: SwiftSyntaxCorrectableRule, ConfigurationProviderR ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/TrailingSemicolonRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/TrailingSemicolonRule.swift index d7299d3bb..c5c993dea 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/TrailingSemicolonRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/TrailingSemicolonRule.swift @@ -24,7 +24,7 @@ public struct TrailingSemicolonRule: SwiftSyntaxCorrectableRule, ConfigurationPr ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/UnavailableConditionRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/UnavailableConditionRule.swift index 2303f0427..f1085b375 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/UnavailableConditionRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/UnavailableConditionRule.swift @@ -71,7 +71,7 @@ public struct UnavailableConditionRule: ConfigurationProviderRule, SwiftSyntaxRu ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { UnavailableConditionRuleVisitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/UnneededBreakInSwitchRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/UnneededBreakInSwitchRule.swift index 4bc227fed..6c83082f2 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/UnneededBreakInSwitchRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/UnneededBreakInSwitchRule.swift @@ -50,7 +50,7 @@ public struct UnneededBreakInSwitchRule: SwiftSyntaxRule, ConfigurationProviderR ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { UnneededBreakInSwitchRuleVisitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/UntypedErrorInCatchRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/UntypedErrorInCatchRule.swift index 1c030b04b..8d4d6c439 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/UntypedErrorInCatchRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/UntypedErrorInCatchRule.swift @@ -87,7 +87,7 @@ public struct UntypedErrorInCatchRule: OptInRule, ConfigurationProviderRule, Swi Example("do {\n try foo() \n} ↓catch (let error) {}"): Example("do {\n try foo() \n} catch {}") ]) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { UntypedErrorInCatchRuleVisitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/VoidFunctionInTernaryConditionRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/VoidFunctionInTernaryConditionRule.swift index b74cffadb..6e96937ba 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/VoidFunctionInTernaryConditionRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/VoidFunctionInTernaryConditionRule.swift @@ -108,7 +108,7 @@ public struct VoidFunctionInTernaryConditionRule: ConfigurationProviderRule, Swi ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { VoidFunctionInTernaryConditionVisitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Idiomatic/XCTFailMessageRule.swift b/Source/SwiftLintFramework/Rules/Idiomatic/XCTFailMessageRule.swift index c4829be8c..9a44f8ca2 100644 --- a/Source/SwiftLintFramework/Rules/Idiomatic/XCTFailMessageRule.swift +++ b/Source/SwiftLintFramework/Rules/Idiomatic/XCTFailMessageRule.swift @@ -36,7 +36,7 @@ public struct XCTFailMessageRule: SwiftSyntaxRule, ConfigurationProviderRule { ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/AnyObjectProtocolRule.swift b/Source/SwiftLintFramework/Rules/Lint/AnyObjectProtocolRule.swift index 1c3aa2f54..11cbf2b14 100644 --- a/Source/SwiftLintFramework/Rules/Lint/AnyObjectProtocolRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/AnyObjectProtocolRule.swift @@ -42,7 +42,7 @@ public struct AnyObjectProtocolRule: SwiftSyntaxCorrectableRule, OptInRule, Conf ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { warnDeprecatedOnce() return Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Lint/ArrayInitRule.swift b/Source/SwiftLintFramework/Rules/Lint/ArrayInitRule.swift index 90484322a..10861106f 100644 --- a/Source/SwiftLintFramework/Rules/Lint/ArrayInitRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/ArrayInitRule.swift @@ -51,7 +51,7 @@ public struct ArrayInitRule: SwiftSyntaxRule, ConfigurationProviderRule, OptInRu ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/ClassDelegateProtocolRule.swift b/Source/SwiftLintFramework/Rules/Lint/ClassDelegateProtocolRule.swift index 2435e2875..7974034ee 100644 --- a/Source/SwiftLintFramework/Rules/Lint/ClassDelegateProtocolRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/ClassDelegateProtocolRule.swift @@ -31,7 +31,7 @@ public struct ClassDelegateProtocolRule: SwiftSyntaxRule, ConfigurationProviderR ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/DeploymentTargetRule.swift b/Source/SwiftLintFramework/Rules/Lint/DeploymentTargetRule.swift index 1942f4b11..5481e5e8d 100644 --- a/Source/SwiftLintFramework/Rules/Lint/DeploymentTargetRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/DeploymentTargetRule.swift @@ -16,7 +16,7 @@ public struct DeploymentTargetRule: ConfigurationProviderRule, SwiftSyntaxRule { triggeringExamples: DeploymentTargetRuleExamples.triggeringExamples ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(platformToConfiguredMinVersion: platformToConfiguredMinVersion) } diff --git a/Source/SwiftLintFramework/Rules/Lint/DuplicateEnumCasesRule.swift b/Source/SwiftLintFramework/Rules/Lint/DuplicateEnumCasesRule.swift index 12afae9c1..70401527b 100644 --- a/Source/SwiftLintFramework/Rules/Lint/DuplicateEnumCasesRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/DuplicateEnumCasesRule.swift @@ -37,7 +37,7 @@ public struct DuplicateEnumCasesRule: ConfigurationProviderRule, SwiftSyntaxRule ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/DynamicInlineRule.swift b/Source/SwiftLintFramework/Rules/Lint/DynamicInlineRule.swift index efac56b27..73c0ca1b4 100644 --- a/Source/SwiftLintFramework/Rules/Lint/DynamicInlineRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/DynamicInlineRule.swift @@ -24,7 +24,7 @@ public struct DynamicInlineRule: SwiftSyntaxRule, ConfigurationProviderRule { ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/EmptyXCTestMethodRule.swift b/Source/SwiftLintFramework/Rules/Lint/EmptyXCTestMethodRule.swift index 67d5592ea..cec2b0105 100644 --- a/Source/SwiftLintFramework/Rules/Lint/EmptyXCTestMethodRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/EmptyXCTestMethodRule.swift @@ -14,7 +14,7 @@ public struct EmptyXCTestMethodRule: OptInRule, ConfigurationProviderRule, Swift triggeringExamples: EmptyXCTestMethodRuleExamples.triggeringExamples ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { EmptyXCTestMethodRuleVisitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/IBInspectableInExtensionRule.swift b/Source/SwiftLintFramework/Rules/Lint/IBInspectableInExtensionRule.swift index 0b6e3c6cb..a31fc0aad 100644 --- a/Source/SwiftLintFramework/Rules/Lint/IBInspectableInExtensionRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/IBInspectableInExtensionRule.swift @@ -26,7 +26,7 @@ public struct IBInspectableInExtensionRule: SwiftSyntaxRule, ConfigurationProvid ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/IdenticalOperandsRule.swift b/Source/SwiftLintFramework/Rules/Lint/IdenticalOperandsRule.swift index e8df3f633..246242c5b 100644 --- a/Source/SwiftLintFramework/Rules/Lint/IdenticalOperandsRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/IdenticalOperandsRule.swift @@ -75,7 +75,7 @@ public struct IdenticalOperandsRule: ConfigurationProviderRule, SwiftSyntaxRule, syntaxTree.folded() } - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/InertDeferRule.swift b/Source/SwiftLintFramework/Rules/Lint/InertDeferRule.swift index 977d4d4b0..f151543ab 100644 --- a/Source/SwiftLintFramework/Rules/Lint/InertDeferRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/InertDeferRule.swift @@ -50,7 +50,7 @@ public struct InertDeferRule: ConfigurationProviderRule, SwiftSyntaxRule { ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/NSLocalizedStringRequireBundleRule.swift b/Source/SwiftLintFramework/Rules/Lint/NSLocalizedStringRequireBundleRule.swift index 40cb9ee3d..0adefb24d 100644 --- a/Source/SwiftLintFramework/Rules/Lint/NSLocalizedStringRequireBundleRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/NSLocalizedStringRequireBundleRule.swift @@ -42,7 +42,7 @@ public struct NSLocalizedStringRequireBundleRule: SwiftSyntaxRule, OptInRule, Co ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/NSObjectPreferIsEqualRule.swift b/Source/SwiftLintFramework/Rules/Lint/NSObjectPreferIsEqualRule.swift index 7d3d8f373..6515d4de6 100644 --- a/Source/SwiftLintFramework/Rules/Lint/NSObjectPreferIsEqualRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/NSObjectPreferIsEqualRule.swift @@ -14,7 +14,7 @@ public struct NSObjectPreferIsEqualRule: SwiftSyntaxRule, ConfigurationProviderR triggeringExamples: NSObjectPreferIsEqualRuleExamples.triggeringExamples ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/PrivateActionRule.swift b/Source/SwiftLintFramework/Rules/Lint/PrivateActionRule.swift index e75f6e075..01ae12512 100644 --- a/Source/SwiftLintFramework/Rules/Lint/PrivateActionRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/PrivateActionRule.swift @@ -33,7 +33,7 @@ public struct PrivateActionRule: SwiftSyntaxRule, OptInRule, ConfigurationProvid ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/PrivateOutletRule.swift b/Source/SwiftLintFramework/Rules/Lint/PrivateOutletRule.swift index 4d29f80ac..c9cc5ea3d 100644 --- a/Source/SwiftLintFramework/Rules/Lint/PrivateOutletRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/PrivateOutletRule.swift @@ -78,7 +78,7 @@ public struct PrivateOutletRule: SwiftSyntaxRule, OptInRule, ConfigurationProvid ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(allowPrivateSet: configuration.allowPrivateSet) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/PrivateUnitTestRule.swift b/Source/SwiftLintFramework/Rules/Lint/PrivateUnitTestRule.swift index f567b2e23..f20636eac 100644 --- a/Source/SwiftLintFramework/Rules/Lint/PrivateUnitTestRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/PrivateUnitTestRule.swift @@ -136,7 +136,7 @@ public struct PrivateUnitTestRule: SwiftSyntaxCorrectableRule, ConfigurationProv ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(parentClassRegex: configuration.regex) } diff --git a/Source/SwiftLintFramework/Rules/Lint/ProhibitedInterfaceBuilderRule.swift b/Source/SwiftLintFramework/Rules/Lint/ProhibitedInterfaceBuilderRule.swift index 3a59b6fda..f68f34e22 100644 --- a/Source/SwiftLintFramework/Rules/Lint/ProhibitedInterfaceBuilderRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/ProhibitedInterfaceBuilderRule.swift @@ -20,7 +20,7 @@ public struct ProhibitedInterfaceBuilderRule: ConfigurationProviderRule, SwiftSy ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/QuickDiscouragedFocusedTestRule.swift b/Source/SwiftLintFramework/Rules/Lint/QuickDiscouragedFocusedTestRule.swift index 8f6fc75ac..1e5012fd6 100644 --- a/Source/SwiftLintFramework/Rules/Lint/QuickDiscouragedFocusedTestRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/QuickDiscouragedFocusedTestRule.swift @@ -14,7 +14,7 @@ public struct QuickDiscouragedFocusedTestRule: OptInRule, ConfigurationProviderR triggeringExamples: QuickDiscouragedFocusedTestRuleExamples.triggeringExamples ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/QuickDiscouragedPendingTestRule.swift b/Source/SwiftLintFramework/Rules/Lint/QuickDiscouragedPendingTestRule.swift index 53f1e651e..952a014d6 100644 --- a/Source/SwiftLintFramework/Rules/Lint/QuickDiscouragedPendingTestRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/QuickDiscouragedPendingTestRule.swift @@ -14,7 +14,7 @@ public struct QuickDiscouragedPendingTestRule: OptInRule, ConfigurationProviderR triggeringExamples: QuickDiscouragedPendingTestRuleExamples.triggeringExamples ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/SelfInPropertyInitializationRule.swift b/Source/SwiftLintFramework/Rules/Lint/SelfInPropertyInitializationRule.swift index 5c8a4c5e2..849837678 100644 --- a/Source/SwiftLintFramework/Rules/Lint/SelfInPropertyInitializationRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/SelfInPropertyInitializationRule.swift @@ -92,7 +92,7 @@ public struct SelfInPropertyInitializationRule: ConfigurationProviderRule, Swift ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/StrongIBOutletRule.swift b/Source/SwiftLintFramework/Rules/Lint/StrongIBOutletRule.swift index 446562d94..6c622568d 100644 --- a/Source/SwiftLintFramework/Rules/Lint/StrongIBOutletRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/StrongIBOutletRule.swift @@ -29,7 +29,7 @@ public struct StrongIBOutletRule: ConfigurationProviderRule, SwiftSyntaxCorrecta ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Lint/UnownedVariableCaptureRule.swift b/Source/SwiftLintFramework/Rules/Lint/UnownedVariableCaptureRule.swift index d05850a43..1dd05e4a3 100644 --- a/Source/SwiftLintFramework/Rules/Lint/UnownedVariableCaptureRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/UnownedVariableCaptureRule.swift @@ -25,7 +25,7 @@ public struct UnownedVariableCaptureRule: SwiftSyntaxRule, OptInRule, Configurat ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { UnownedVariableCaptureRuleVisitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/UnusedClosureParameterRule.swift b/Source/SwiftLintFramework/Rules/Lint/UnusedClosureParameterRule.swift index 6fc5d9746..aedef05e7 100644 --- a/Source/SwiftLintFramework/Rules/Lint/UnusedClosureParameterRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/UnusedClosureParameterRule.swift @@ -16,7 +16,7 @@ public struct UnusedClosureParameterRule: SwiftSyntaxCorrectableRule, Configurat corrections: UnusedClosureParameterRuleExamples.corrections ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Lint/UnusedSetterValueRule.swift b/Source/SwiftLintFramework/Rules/Lint/UnusedSetterValueRule.swift index 0414e861c..3193ccfe4 100644 --- a/Source/SwiftLintFramework/Rules/Lint/UnusedSetterValueRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/UnusedSetterValueRule.swift @@ -129,7 +129,7 @@ public struct UnusedSetterValueRule: ConfigurationProviderRule, SwiftSyntaxRule ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Lint/ValidIBInspectableRule.swift b/Source/SwiftLintFramework/Rules/Lint/ValidIBInspectableRule.swift index 7074cbd4c..4aecda9c2 100644 --- a/Source/SwiftLintFramework/Rules/Lint/ValidIBInspectableRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/ValidIBInspectableRule.swift @@ -110,7 +110,7 @@ public struct ValidIBInspectableRule: SwiftSyntaxRule, ConfigurationProviderRule ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Lint/YodaConditionRule.swift b/Source/SwiftLintFramework/Rules/Lint/YodaConditionRule.swift index ad06c5ee0..d5f94928f 100644 --- a/Source/SwiftLintFramework/Rules/Lint/YodaConditionRule.swift +++ b/Source/SwiftLintFramework/Rules/Lint/YodaConditionRule.swift @@ -36,7 +36,7 @@ public struct YodaConditionRule: OptInRule, ConfigurationProviderRule, SwiftSynt Example("if ↓200 <= i && i <= 299 || ↓600 <= i {}") ]) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { YodaConditionRuleVisitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Performance/ContainsOverFilterCountRule.swift b/Source/SwiftLintFramework/Rules/Performance/ContainsOverFilterCountRule.swift index 30ae8873d..11080316b 100644 --- a/Source/SwiftLintFramework/Rules/Performance/ContainsOverFilterCountRule.swift +++ b/Source/SwiftLintFramework/Rules/Performance/ContainsOverFilterCountRule.swift @@ -30,7 +30,7 @@ public struct ContainsOverFilterCountRule: SwiftSyntaxRule, OptInRule, Configura } ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Performance/ContainsOverFilterIsEmptyRule.swift b/Source/SwiftLintFramework/Rules/Performance/ContainsOverFilterIsEmptyRule.swift index 248b4538f..0d3699f70 100644 --- a/Source/SwiftLintFramework/Rules/Performance/ContainsOverFilterIsEmptyRule.swift +++ b/Source/SwiftLintFramework/Rules/Performance/ContainsOverFilterIsEmptyRule.swift @@ -28,7 +28,7 @@ public struct ContainsOverFilterIsEmptyRule: SwiftSyntaxRule, OptInRule, Configu ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Performance/ContainsOverRangeNilComparisonRule.swift b/Source/SwiftLintFramework/Rules/Performance/ContainsOverRangeNilComparisonRule.swift index 08f81d638..444c2e0b5 100644 --- a/Source/SwiftLintFramework/Rules/Performance/ContainsOverRangeNilComparisonRule.swift +++ b/Source/SwiftLintFramework/Rules/Performance/ContainsOverRangeNilComparisonRule.swift @@ -27,7 +27,7 @@ public struct ContainsOverRangeNilComparisonRule: SwiftSyntaxRule, OptInRule, Co syntaxTree.folded() } - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Performance/EmptyCollectionLiteralRule.swift b/Source/SwiftLintFramework/Rules/Performance/EmptyCollectionLiteralRule.swift index 4dff56c70..80ec77399 100644 --- a/Source/SwiftLintFramework/Rules/Performance/EmptyCollectionLiteralRule.swift +++ b/Source/SwiftLintFramework/Rules/Performance/EmptyCollectionLiteralRule.swift @@ -28,7 +28,7 @@ public struct EmptyCollectionLiteralRule: SwiftSyntaxRule, ConfigurationProvider ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Performance/EmptyStringRule.swift b/Source/SwiftLintFramework/Rules/Performance/EmptyStringRule.swift index 1f390142e..e85821cb3 100644 --- a/Source/SwiftLintFramework/Rules/Performance/EmptyStringRule.swift +++ b/Source/SwiftLintFramework/Rules/Performance/EmptyStringRule.swift @@ -24,7 +24,7 @@ public struct EmptyStringRule: ConfigurationProviderRule, OptInRule, SwiftSyntax ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Performance/FirstWhereRule.swift b/Source/SwiftLintFramework/Rules/Performance/FirstWhereRule.swift index 474036a2c..23e6c788b 100644 --- a/Source/SwiftLintFramework/Rules/Performance/FirstWhereRule.swift +++ b/Source/SwiftLintFramework/Rules/Performance/FirstWhereRule.swift @@ -32,7 +32,7 @@ public struct FirstWhereRule: SwiftSyntaxRule, OptInRule, ConfigurationProviderR ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Performance/FlatMapOverMapReduceRule.swift b/Source/SwiftLintFramework/Rules/Performance/FlatMapOverMapReduceRule.swift index 203cca4c8..6eab9d4ea 100644 --- a/Source/SwiftLintFramework/Rules/Performance/FlatMapOverMapReduceRule.swift +++ b/Source/SwiftLintFramework/Rules/Performance/FlatMapOverMapReduceRule.swift @@ -19,7 +19,7 @@ public struct FlatMapOverMapReduceRule: SwiftSyntaxRule, OptInRule, Configuratio ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Performance/LastWhereRule.swift b/Source/SwiftLintFramework/Rules/Performance/LastWhereRule.swift index dcd5b19e1..54a95d7ec 100644 --- a/Source/SwiftLintFramework/Rules/Performance/LastWhereRule.swift +++ b/Source/SwiftLintFramework/Rules/Performance/LastWhereRule.swift @@ -28,7 +28,7 @@ public struct LastWhereRule: SwiftSyntaxRule, OptInRule, ConfigurationProviderRu ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Performance/ReduceBooleanRule.swift b/Source/SwiftLintFramework/Rules/Performance/ReduceBooleanRule.swift index 9826f54d7..fd1f9abdc 100644 --- a/Source/SwiftLintFramework/Rules/Performance/ReduceBooleanRule.swift +++ b/Source/SwiftLintFramework/Rules/Performance/ReduceBooleanRule.swift @@ -26,7 +26,7 @@ public struct ReduceBooleanRule: SwiftSyntaxRule, ConfigurationProviderRule { ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Performance/SortedFirstLastRule.swift b/Source/SwiftLintFramework/Rules/Performance/SortedFirstLastRule.swift index a3342c7a8..1401456ad 100644 --- a/Source/SwiftLintFramework/Rules/Performance/SortedFirstLastRule.swift +++ b/Source/SwiftLintFramework/Rules/Performance/SortedFirstLastRule.swift @@ -42,7 +42,7 @@ public struct SortedFirstLastRule: SwiftSyntaxRule, OptInRule, ConfigurationProv ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Style/AttributesRule.swift b/Source/SwiftLintFramework/Rules/Style/AttributesRule.swift index b66864848..7a309db81 100644 --- a/Source/SwiftLintFramework/Rules/Style/AttributesRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/AttributesRule.swift @@ -17,7 +17,7 @@ public struct AttributesRule: SwiftSyntaxRule, OptInRule, ConfigurationProviderR triggeringExamples: AttributesRuleExamples.triggeringExamples ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor( locationConverter: file.locationConverter, configuration: configuration diff --git a/Source/SwiftLintFramework/Rules/Style/ClosingBraceRule.swift b/Source/SwiftLintFramework/Rules/Style/ClosingBraceRule.swift index 5ac533ddc..2024aff65 100644 --- a/Source/SwiftLintFramework/Rules/Style/ClosingBraceRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/ClosingBraceRule.swift @@ -23,7 +23,7 @@ public struct ClosingBraceRule: SwiftSyntaxCorrectableRule, ConfigurationProvide ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Style/ClosureParameterPositionRule.swift b/Source/SwiftLintFramework/Rules/Style/ClosureParameterPositionRule.swift index d4f20420a..6e7288bc5 100644 --- a/Source/SwiftLintFramework/Rules/Style/ClosureParameterPositionRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/ClosureParameterPositionRule.swift @@ -96,7 +96,7 @@ public struct ClosureParameterPositionRule: SwiftSyntaxRule, ConfigurationProvid ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(locationConverter: file.locationConverter) } } diff --git a/Source/SwiftLintFramework/Rules/Style/ClosureSpacingRule.swift b/Source/SwiftLintFramework/Rules/Style/ClosureSpacingRule.swift index 88def8656..c854e0e5b 100644 --- a/Source/SwiftLintFramework/Rules/Style/ClosureSpacingRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/ClosureSpacingRule.swift @@ -38,7 +38,7 @@ public struct ClosureSpacingRule: SwiftSyntaxCorrectableRule, ConfigurationProvi ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { ClosureSpacingRuleVisitor(locationConverter: file.locationConverter) } diff --git a/Source/SwiftLintFramework/Rules/Style/ComputedAccessorsOrderRule.swift b/Source/SwiftLintFramework/Rules/Style/ComputedAccessorsOrderRule.swift index 5c11bf9d6..f101e9a3a 100644 --- a/Source/SwiftLintFramework/Rules/Style/ComputedAccessorsOrderRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/ComputedAccessorsOrderRule.swift @@ -14,7 +14,7 @@ public struct ComputedAccessorsOrderRule: ConfigurationProviderRule, SwiftSyntax triggeringExamples: ComputedAccessorsOrderRuleExamples.triggeringExamples ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { ComputedAccessorsOrderRuleVisitor(expectedOrder: configuration.order) } } diff --git a/Source/SwiftLintFramework/Rules/Style/ConditionalReturnsOnNewlineRule.swift b/Source/SwiftLintFramework/Rules/Style/ConditionalReturnsOnNewlineRule.swift index 5273dfeb5..ebd8a0679 100644 --- a/Source/SwiftLintFramework/Rules/Style/ConditionalReturnsOnNewlineRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/ConditionalReturnsOnNewlineRule.swift @@ -34,7 +34,7 @@ public struct ConditionalReturnsOnNewlineRule: ConfigurationProviderRule, OptInR ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor( ifOnly: configuration.ifOnly, locationConverter: file.locationConverter diff --git a/Source/SwiftLintFramework/Rules/Style/EmptyEnumArgumentsRule.swift b/Source/SwiftLintFramework/Rules/Style/EmptyEnumArgumentsRule.swift index fa05e6b65..3172ed643 100644 --- a/Source/SwiftLintFramework/Rules/Style/EmptyEnumArgumentsRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/EmptyEnumArgumentsRule.swift @@ -107,7 +107,7 @@ public struct EmptyEnumArgumentsRule: SwiftSyntaxCorrectableRule, ConfigurationP ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Style/EmptyParametersRule.swift b/Source/SwiftLintFramework/Rules/Style/EmptyParametersRule.swift index 3285069f4..f007ba107 100644 --- a/Source/SwiftLintFramework/Rules/Style/EmptyParametersRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/EmptyParametersRule.swift @@ -33,7 +33,7 @@ public struct EmptyParametersRule: SwiftSyntaxCorrectableRule, ConfigurationProv ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Style/EmptyParenthesesWithTrailingClosureRule.swift b/Source/SwiftLintFramework/Rules/Style/EmptyParenthesesWithTrailingClosureRule.swift index 88a958ebd..f4d3d2f55 100644 --- a/Source/SwiftLintFramework/Rules/Style/EmptyParenthesesWithTrailingClosureRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/EmptyParenthesesWithTrailingClosureRule.swift @@ -46,7 +46,7 @@ public struct EmptyParenthesesWithTrailingClosureRule: SwiftSyntaxCorrectableRul ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Style/ImplicitGetterRule.swift b/Source/SwiftLintFramework/Rules/Style/ImplicitGetterRule.swift index 5b0a18bd6..7156a3532 100644 --- a/Source/SwiftLintFramework/Rules/Style/ImplicitGetterRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/ImplicitGetterRule.swift @@ -14,7 +14,7 @@ public struct ImplicitGetterRule: ConfigurationProviderRule, SwiftSyntaxRule { triggeringExamples: ImplicitGetterRuleExamples.triggeringExamples ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { ImplicitGetterRuleVisitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Style/MultipleClosuresWithTrailingClosureRule.swift b/Source/SwiftLintFramework/Rules/Style/MultipleClosuresWithTrailingClosureRule.swift index 625699366..255a7d8ae 100644 --- a/Source/SwiftLintFramework/Rules/Style/MultipleClosuresWithTrailingClosureRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/MultipleClosuresWithTrailingClosureRule.swift @@ -37,7 +37,7 @@ public struct MultipleClosuresWithTrailingClosureRule: SwiftSyntaxRule, Configur ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Style/NoSpaceInMethodCallRule.swift b/Source/SwiftLintFramework/Rules/Style/NoSpaceInMethodCallRule.swift index ef4e0af8f..8b25350d1 100644 --- a/Source/SwiftLintFramework/Rules/Style/NoSpaceInMethodCallRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/NoSpaceInMethodCallRule.swift @@ -45,7 +45,7 @@ public struct NoSpaceInMethodCallRule: SwiftSyntaxCorrectableRule, Configuration ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Style/NumberSeparatorRule.swift b/Source/SwiftLintFramework/Rules/Style/NumberSeparatorRule.swift index 2ffbe8752..1436e4a2d 100644 --- a/Source/SwiftLintFramework/Rules/Style/NumberSeparatorRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/NumberSeparatorRule.swift @@ -20,7 +20,7 @@ public struct NumberSeparatorRule: OptInRule, SwiftSyntaxCorrectableRule, Config corrections: NumberSeparatorRuleExamples.corrections ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(configuration: configuration) } diff --git a/Source/SwiftLintFramework/Rules/Style/OperatorFunctionWhitespaceRule.swift b/Source/SwiftLintFramework/Rules/Style/OperatorFunctionWhitespaceRule.swift index 356c5dbed..618b908f3 100644 --- a/Source/SwiftLintFramework/Rules/Style/OperatorFunctionWhitespaceRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/OperatorFunctionWhitespaceRule.swift @@ -25,7 +25,7 @@ public struct OperatorFunctionWhitespaceRule: ConfigurationProviderRule, SwiftSy ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Style/ProtocolPropertyAccessorsOrderRule.swift b/Source/SwiftLintFramework/Rules/Style/ProtocolPropertyAccessorsOrderRule.swift index 5e86f3ea4..538d16d16 100644 --- a/Source/SwiftLintFramework/Rules/Style/ProtocolPropertyAccessorsOrderRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/ProtocolPropertyAccessorsOrderRule.swift @@ -24,7 +24,7 @@ public struct ProtocolPropertyAccessorsOrderRule: ConfigurationProviderRule, Swi ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Style/RedundantDiscardableLetRule.swift b/Source/SwiftLintFramework/Rules/Style/RedundantDiscardableLetRule.swift index 1bb84e9ea..f4797d9d2 100644 --- a/Source/SwiftLintFramework/Rules/Style/RedundantDiscardableLetRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/RedundantDiscardableLetRule.swift @@ -28,7 +28,7 @@ public struct RedundantDiscardableLetRule: SwiftSyntaxCorrectableRule, Configura ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } diff --git a/Source/SwiftLintFramework/Rules/Style/SelfBindingRule.swift b/Source/SwiftLintFramework/Rules/Style/SelfBindingRule.swift index b233c3e02..746e02b12 100644 --- a/Source/SwiftLintFramework/Rules/Style/SelfBindingRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/SelfBindingRule.swift @@ -42,7 +42,7 @@ public struct SelfBindingRule: SwiftSyntaxCorrectableRule, ConfigurationProvider ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { SelfBindingRuleVisitor(bindIdentifier: configuration.bindIdentifier) } diff --git a/Source/SwiftLintFramework/Rules/Style/ShorthandOperatorRule.swift b/Source/SwiftLintFramework/Rules/Style/ShorthandOperatorRule.swift index e6d43de35..a6cdbf04e 100644 --- a/Source/SwiftLintFramework/Rules/Style/ShorthandOperatorRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/ShorthandOperatorRule.swift @@ -46,7 +46,7 @@ public struct ShorthandOperatorRule: ConfigurationProviderRule, SwiftSyntaxRule syntaxTree.folded() } - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) } } diff --git a/Source/SwiftLintFramework/Rules/Style/SwitchCaseOnNewlineRule.swift b/Source/SwiftLintFramework/Rules/Style/SwitchCaseOnNewlineRule.swift index dedc2258f..1ecc472c4 100644 --- a/Source/SwiftLintFramework/Rules/Style/SwitchCaseOnNewlineRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/SwitchCaseOnNewlineRule.swift @@ -62,7 +62,7 @@ public struct SwitchCaseOnNewlineRule: SwiftSyntaxRule, ConfigurationProviderRul ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(locationConverter: file.locationConverter) } } diff --git a/Source/SwiftLintFramework/Rules/Style/UnneededParenthesesInClosureArgumentRule.swift b/Source/SwiftLintFramework/Rules/Style/UnneededParenthesesInClosureArgumentRule.swift index b4a7d7905..bb97bf8b5 100644 --- a/Source/SwiftLintFramework/Rules/Style/UnneededParenthesesInClosureArgumentRule.swift +++ b/Source/SwiftLintFramework/Rules/Style/UnneededParenthesesInClosureArgumentRule.swift @@ -75,7 +75,7 @@ public struct UnneededParenthesesInClosureArgumentRule: ConfigurationProviderRul ] ) - public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor? { + public func makeVisitor(file: SwiftLintFile) -> ViolationsSyntaxVisitor { Visitor(viewMode: .sourceAccurate) }