SwiftLintBot comments

Results 110 comments of


                                            SwiftLintBot

Add `scope_depth` rule to validate that scopes aren't nested more than `N` levels deep

822 Warnings :warning: This PR introduced a violation in Aerial: /Aerial/Source/Models/API/Forecast.swift:197:29: warning: Scope Depth Violation: Exceeds configured scope depth: warningDepth: 7, errorDepth: 10 (scope_depth) :warning: This PR introduced a violation...

Rewrite RedundantVoidReturnRule with SwiftSyntax

261 Warnings :warning: This PR may need tests. :warning: This PR introduced a violation in Aerial: /Aerial/Source/Models/Cache/VideoCache.swift:358:38: warning: Redundant Void Return Violation: Returning Void in a function declaration is redundant...

Fix excluded path calculation when passing in individual files

12 Messages :book: Linting Aerial with this PR took 2.05s vs 2.0s on master (2% slower) :book: Linting Alamofire with this PR took 2.86s vs 2.94s on master (2% faster)...

Rewrite `for_where`, adding `allow_for_as_filter` config

1 Warning :warning: This PR may need tests. Generated by :no_entry_sign: Danger

Enable caching for collecting rules

12 Messages :book: Linting Aerial with this PR took 0.96s vs 0.95s on master (1% slower) :book: Linting Alamofire with this PR took 1.01s vs 1.0s on master (1% slower)...

Allow opening brace on newline in case of multiline class or struct

1 Warning :warning: Please include a CHANGELOG entry to credit yourself! You can find it at CHANGELOG.md. 12 Messages :book: Linting Aerial with this PR took 2.42s vs 2.49s on...

Rewrite `private_outlet` with SwiftSyntax

2 Warnings :warning: If this is a user-facing change, please include a CHANGELOG entry to credit yourself! You can find it at CHANGELOG.md. :warning: This PR may need tests. 12...

Rewrite `fatal_error_message` rule with SwiftSyntax

22 Warnings :warning: If this is a user-facing change, please include a CHANGELOG entry to credit yourself! You can find it at CHANGELOG.md. :warning: This PR introduced a violation in...

Rewrite `ibinspectable_in_extension` with SwiftSyntax

4 Warnings :warning: If this is a user-facing change, please include a CHANGELOG entry to credit yourself! You can find it at CHANGELOG.md. :warning: This PR may need tests. :warning:...

Ignore comments starting right at the beginning of a line

102 Warnings :warning: This PR introduced a violation in Firefox: /Client/Nimbus/NimbusFeatureFlagLayer.swift:48:1: warning: Indentation Width Violation: Code should be indented using one tab or 4 spaces. (indentation_width) :warning: This PR introduced...