Danny Mösch

Closing this as completed. The remaining part "implicit `self` in classes when the closure isn't `@escaping`" is requested by #4964.

Closing in favor of #4655.

Thinking about this issue again, while my previous statement about a new Analyzer rule remains true to really hit all cases where `Self` is allowed, skipping extensions entirely is actually...

Rebased and updated the PR.

Hey @agustr! I like your idea. Are you interested in continuing on this (pretty old) PR? If so I would have some review comments. 😬

You need to add `SwiftyTextTable` to the `frameworkDependencies` in `Package.swift` to make the builds succeed. Could you please also add some tests for the new reporter?

> I dont understand what causes the failed checks. > I am willing to fix it but dont know where to start looking for the trouble. Some of the necessary...

We just have #4820 merged. Is it sufficient for you, @agustr?

I'm willing to move this forward. You too, @m-chojnacki? If so, let's start with rebasing the changes onto the current `main` branch.

> Hey @SimplyDanny, I am willing to move this forward and rebase it, I'm just worried how to handle cases mentioned in [#3905 (comment)](https://github.com/realm/SwiftLint/pull/3905#issuecomment-1199085814) Well, I agree this rule will...