Changing reason message

realm · Jan 15, 2016 · 75a4f2f · 75a4f2f
1 parent 12eb3fe
commit 75a4f2f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/Source/SwiftLintFramework/Rules/FunctionBodyLengthRule.swift b/Source/SwiftLintFramework/Rules/FunctionBodyLengthRule.swift
@@ -85,9 +85,9 @@ public struct FunctionBodyLengthRule: ASTRule, ViolationLevelRule {
                         return [StyleViolation(ruleDescription: self.dynamicType.description,
                             severity: parameter.severity,
                             location: location,
-                            reason: "Function body should span \(parameter.value) lines " +
-                            "or less: currently spans \(lineCount) lines (already ignoring " +
-                            "comment and whitespace only ones)")]
+                            reason: "Function body should span \(parameter.value) lines or less " +
+                            "excluding comments and whitespace: currently spans \(lineCount) " +
+                            "lines")]
                     }
 
                 }

diff --git a/Source/SwiftLintFrameworkTests/FunctionBodyLengthRuleTests.swift b/Source/SwiftLintFrameworkTests/FunctionBodyLengthRuleTests.swift
@@ -24,8 +24,8 @@ class FunctionBodyLengthRuleTests: XCTestCase {
         XCTAssertEqual(violations(longerFunctionBody), [StyleViolation(
             ruleDescription: FunctionBodyLengthRule.description,
             location: Location(file: nil, line: 1, character: 1),
-            reason: "Function body should span 40 lines or less: currently spans 41 lines " +
-            "(already ignoring comment and whitespace only ones)")])
+            reason: "Function body should span 40 lines or less excluding comments and " +
+            "whitespace: currently spans 41 lines")])
 
         let longerFunctionBodyWithEmptyLines = "func abc() {" +
             Repeat(count: 100, repeatedValue: "\n").joinWithSeparator("") +
@@ -49,8 +49,8 @@ class FunctionBodyLengthRuleTests: XCTestCase {
         XCTAssertEqual(violations(longerFunctionBodyWithComments), [StyleViolation(
             ruleDescription: FunctionBodyLengthRule.description,
             location: Location(file: nil, line: 1, character: 1),
-            reason: "Function body should span 40 lines or less: currently spans 41 lines " +
-            "(already ignoring comment and whitespace only ones)")])
+            reason: "Function body should span 40 lines or less excluding comments and " +
+            "whitespace: currently spans 41 lines")])
     }
 
     func testFunctionBodyLengthsWithMultilineComments() {
@@ -69,7 +69,7 @@ class FunctionBodyLengthRuleTests: XCTestCase {
         XCTAssertEqual(violations(longerFunctionBodyWithMultilineComments), [StyleViolation(
             ruleDescription: FunctionBodyLengthRule.description,
             location: Location(file: nil, line: 1, character: 1),
-            reason: "Function body should span 40 lines or less: currently spans 41 lines " +
-            "(already ignoring comment and whitespace only ones)")])
+            reason: "Function body should span 40 lines or less excluding comments and " +
+            "whitespace: currently spans 41 lines")])
     }
 }