André Pacheco Neves
André Pacheco Neves
I can confirm that SwiftLint seems to be doing some processing for every `excluded:` path, leading to degraded performance in large folders (like `.build/`), when ideally it should be able...
For completeness' sake, following a discussion in this [PF's slack thread](https://pointfreecommunity.slack.com/archives/C04L2D0MNJH/p1693911737867579), to ensure `CLLocationManager` is accessed from the main `RunLoop` an alternative solution could be to use a `MainActorIsolated` structure...
@rcasula can this MR be marked as ready for review? What documentation updates are missing? Thanks!
But according to the rule's [documentation](https://realm.github.io/SwiftLint/prefer_key_path.html), should `{ $0 }` trigger a warning or not? I confess I'm a bit confused... 🤔 This clearly states that `{ $0 }` should...
Thank you very much for clarifying 🙏🏼 Would it make sense to add a new configuration that would allow ignoring the identity closure `{ $0 }` instead of requiring `(\.self)`,...
I was suggesting this new configuration assuming we could have the rule working consistently across versions, but also as an extra knob if users want to explicitly ignore `{ $0...
Oh, I understand now. Was missing the fact that `\.self` didn't work at all in Swift 5. 🙈 Thanks for the extra clarification 🙏🏼 > The only thing reasonable would...
@SimplyDanny You were right. I think I got it: #6068 Thanks! 🙏🏼
Ready for re-review @SimplyDanny Not sure what is causing the [buildkite/swiftlint/bazel](https://buildkite.com/swiftlint/swiftlint/builds/10150#01987f1f-21d8-4435-a33a-ed4636a8e026) and [buildkite/swiftlint/registration](https://buildkite.com/swiftlint/swiftlint/builds/10150#01987f1f-21dc-4787-a54a-ac47bf486bc3) CI steps to fail, as I ran both command sequences locally and they succeed. They seem to...