Make unneeded_parentheses_in_closure_argument opt-in

realm · Jul 19, 2017 · 57f52b6 · 57f52b6
1 parent 2eb73a9
commit 57f52b6
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/.swiftlint.yml b/.swiftlint.yml
@@ -23,6 +23,7 @@ opt_in_rules:
   - fatal_error_message
   - vertical_parameter_alignment_on_call
   - let_var_whitespace
+  - unneeded_parentheses_in_closure_argument
 
 file_header:
   required_pattern: |

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -59,8 +59,8 @@
   [Marcelo Fabri](https://github.com/marcelofabri)
   [#1647](https://github.com/realm/SwiftLint/issues/1647)
 
-* Add `unneeded_parentheses_in_closure_argument` correctable rule that warns
-  against using parentheses around argument declarations in closures.  
+* Add `unneeded_parentheses_in_closure_argument` opt-in correctable rule that
+  warns against using parentheses around argument declarations in closures.  
   [Marcelo Fabri](https://github.com/marcelofabri)
   [#1483](https://github.com/realm/SwiftLint/issues/1483)
 

diff --git a/Rules.md b/Rules.md
@@ -12141,7 +12141,7 @@ protocol Foo {
 
 Identifier | Enabled by default | Supports autocorrection | Kind 
 --- | --- | --- | ---
-`unneeded_parentheses_in_closure_argument` | Enabled | Yes | style
+`unneeded_parentheses_in_closure_argument` | Disabled | Yes | style
 
 Parentheses are not needed when declaring closure arguments.
 

diff --git a/Source/SwiftLintFramework/Rules/UnneededParenthesesInClosureArgumentRule.swift b/Source/SwiftLintFramework/Rules/UnneededParenthesesInClosureArgumentRule.swift
@@ -10,7 +10,7 @@ import Foundation
 import Foundation
 import SourceKittenFramework
 
-public struct UnneededParenthesesInClosureArgumentRule: ConfigurationProviderRule, CorrectableRule {
+public struct UnneededParenthesesInClosureArgumentRule: ConfigurationProviderRule, CorrectableRule, OptInRule {
     public var configuration = SeverityConfiguration(.warning)
 
     public init() {}