Feature/sc rules update

resources/swiftlint.yml

@@ -42,41 +42,43 @@ custom_rules:
     regex: "@objcMembers"
     message: "Explicitly use @objc on each member you want to expose to Objective-C"
     severity: error
+  no_direct_standard_out_logs:
+    name: "Writing log messages directly to standard out is disallowed"
+    regex: "(\\bprint|\\bdebugPrint|\\bdump|Swift\\.print|Swift\\.debugPrint|Swift\\.dump)\\s*\\("
+    match_kinds:
+    - identifier
+    message: "Don't commit `print(…)`, `debugPrint(…)`, or `dump(…)` as they write to standard out in release. Either log to a dedicated logging system or silence this warning in debug-only scenarios explicitly using `// swiftlint:disable:next no_direct_standard_out_logs`"
+    severity: warning
 
-colon:
-  apply_to_dictionaries: false
+  colon:
+    apply_to_dictionaries: false
 
-line_length:
+  line_length:
     warning: 400
     error: 500
     ignores_function_declarations: true
     ignores_comments: true
     ignores_urls: true
 
-function_body_length:
+  function_body_length:
     warning: 300
-    error: 500
 
-function_parameter_count:
+  function_parameter_count:
     warning: 6
-    error: 8
 
-type_body_length:
+  type_body_length:
     warning: 300
-    error: 500
 
-file_length:
+  file_length:
     warning: 1000
-    error: 1500
     ignore_comment_only_lines: true
 
-cyclomatic_complexity:
+  cyclomatic_complexity:
     warning: 25
-    error: 25
 
-type_name:
-  max_length:
-    warning: 60
+  type_name:
+    max_length:
+      warning: 60
 
-attributes:
-  always_on_line_above: ["@testable", "@available", "@discardableResult", "@objc", "@IBAction", "@IBDesignable"]
+  attributes:
+    always_on_line_above: ["@testable", "@available", "@discardableResult", "@objc", "@IBAction", "@IBDesignable"]