Generated by 🚫Danger

what needs to happen to get this merged? Is thing we can do to help on this issue. It would be amazing to have cognitive complexity rule enforcement in this library.

also why are these warnings being generated here when none of these projects have anything to do with this project?

what needs to happen to get this merged? Is thing we can do to help on this issue. It would be amazing to have cognitive complexity rule enforcement in this library.

So it will need to be code reviewed and approved, and then merged. Everyone who contributes to SwiftLint works on a volunteer basis, so it can take a while for people to find time to come back to it. If you look at previously merged PRs, you'll see that it can take a while.

I do have some comments on the PR, but I don't have time right now to take a proper look.

Generally if you can keep the branch clean and without merge conflicts that is good. The changelog can be a bit of a hotspot.

also why are these warnings being generated here when none of these projects have anything to do with this project?

So PRs that change any code in SwiftLint (some PRs are excluded but I can't remember the exact basis offhand) are run against a set of popular open source projects, and their violations compared against whatmain would detect.

This gives a kind of sanity check across a much wider codebase, and inspecting the new violations can be a big clue as to where we might be picking up false alarms, and "fixed" violations can be inspected to see if they really should have been fixed by the changes. In this case, it's a new rule, so you'd only expect new violations.