From 9dee92ef6fe8a2a8b458b85457a74e1deeda0b70 Mon Sep 17 00:00:00 2001 From: Maxim Sorokin Date: Fri, 10 Apr 2020 13:34:40 +0300 Subject: [PATCH] added new swiftlint rules --- xcode/.swiftlint.yml | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/xcode/.swiftlint.yml b/xcode/.swiftlint.yml index c61442e..799ada0 100644 --- a/xcode/.swiftlint.yml +++ b/xcode/.swiftlint.yml @@ -207,6 +207,20 @@ 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 + + boolean_redundant_condition: + name: "Redundant Boolean Condition" + regex: "(== true)|(== false)|(!= true)|(!= false)||(\\? true \\: false)|(\\? false \\: true)" + message: "Comparing a boolean to true is redundant (use `?? false` for optionals), and `!`-syntax is preferred over comparing to false." + severity: error + # Rx unused_map_parameter: @@ -222,17 +236,3 @@ custom_rules: regex: '(^\s*\w*\.?)(addSubview)\([\w]+\)\n\1\2' message: "Replace multiple addSubview calls with single addSubviews." severity: warning - - 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 - - boolean_redundant_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