Merge pull request #178 from TouchInstinct/swiftlint_rules

Swiftlint rules

xcode/.swiftlint.yml

@@ -207,6 +207,26 @@ custom_rules:
       - string_interpolation_anchor
       - typeidentifier
 
+  getter_setter_style:
+    name: "Wrong getter/setter code style"
+    regex: "(get|set|willSet|didSet) \\{ [.]*"
+    match_kinds:
+    - keyword
+    message: "Make a new line break when use getter or setter"
+    severity: error
+
+  redundant_boolean_condition:
+    name: "Redundant Boolean Condition"
+    regex: "(== true)|(== false)|(!= true)|(!= false)"
+    message: "Comparing a boolean to true is redundant (use `?? false` for optionals), and `!`-syntax is preferred over comparing to false."
+    severity: error
+
+  redundant_ternary_operator:
+    name: "Redundant Ternary Operator"
+    regex: "(\\? true \\: false)|(\\? false \\: true)"
+    message: "Returning a boolean as true is redundant, and `!`-syntax is preferred over returning as false."
+    severity: error
+
   # Rx
 
   unused_map_parameter:
@@ -222,4 +242,3 @@ custom_rules:
     regex: '(^\s*\w*\.?)(addSubview)\([\w]+\)\n\1\2'
     message: "Replace multiple addSubview calls with single addSubviews."
     severity: warning
-