Add options to exclude declaration types in VerticalWhitespaceOpeningBracesRule

[SpacyRicochet]

In our team, we prefer a variant to this, but only for computed variables and functions. It would be nice if we could have options to exclude types of declarations.

#1518 (comment)

// ❌ not acceptable
var foo: Foo {
    return Foo()
}

// ❌ not acceptable
func foo() -> Foo {

    return Foo()
}

// ✅ acceptable
protocol FooProvider {

    var foo: Foo { get }

}

// ✅ acceptable
class Foo {

    var foo: Foo {
        Foo()
    }

}

[SpacyRicochet]

I did try to alter the rule regex in a custom rule to allow for this, but I haven't managed to get one that checks for var and func or for either of them. My regex fu is not good enough for this, apparently. 😬

[jpsim]

Ideally we'd avoid using regular expressions altogether for this. They've been useful crutches to allow a lot of rules to be added quickly in SwiftLint's early days, but there are much better tools available to accomplish the same thing, in many cases more reliably.

[stale]

This issue has been automatically marked as stale because it has not had any recent activity. Please comment to prevent this issue from being closed. Thank you for your contributions!