Danny Mösch
Danny Mösch
I think we can go with a new option. Having it in the documentation should already make a bit clearer what the idea of the rule is.
You need to rebase to get rid of the failing test.
> > You need to rebase to get rid of the failing test. > > > > I have synced the `main` branch from the original repo into my fork...
This is a quite uncommon use case indeed. Currently, the rule is not configurable at all.
@mildm8nnered: I would like to hear your opinion on this. Is the presentation style clear enough?
> > @mildm8nnered: I would like to hear your opinion on this. Is the presentation style clear enough? > > So I think it's a matter of taste, but I'd...
Considering that SwiftLint is a tool that only comes into play at build time, I don't think this topic is relevant for it. It's not an SDK that gets included...
Looking at the OSS findings, most of them look valid. However, [there are cases](https://github.com/wordpress-mobile/WordPress-iOS/blob/12065e659d39e59a7e9a01a45caf31112dee49b8/WordPress/Classes/ViewRelated/Gutenberg/Collapsable%20Header/Collabsable%20Header%20Filter%20Bar/CollabsableHeaderFilterCollectionViewCell.swift#L83:26) where a sequence of `MemberAccessExpression`s leads to the final element/function. An easy example would be `let...
> > Looking at the OSS findings, most of them look valid. > > However, [there are cases](https://github.com/wordpress-mobile/WordPress-iOS/blob/12065e659d39e59a7e9a01a45caf31112dee49b8/WordPress/Classes/ViewRelated/Gutenberg/Collapsable%20Header/Collabsable%20Header%20Filter%20Bar/CollabsableHeaderFilterCollectionViewCell.swift#L83:26) where a sequence of MemberAccessExpressions leads to the final element/function. An easy...
> That depends on what the previous implementation did. I guess, it triggered, hence the rewritten rule should trigger too. Rebased and addressed this point.