change excludingSyntaxKinds for VerticalWhitespaceBetweenCasesRule.swift

[ilendemli]

Hi I was applying this ruleset and saw it skip some parts of my code

Regarding this:

SwiftLint/Source/SwiftLintBuiltInRules/Rules/Style/VerticalWhitespaceBetweenCasesRule.swift

Line 6 in b42f6ff

return match(pattern: pattern, excludingSyntaxKinds: SyntaxKind.commentAndStringKinds)

I think it should also apply for String cases and only exclude SyntaxKind.commentKinds

[SimplyDanny]

What's an example that would work correctly with this change?

[ilendemli]

Currently running the fix argument against a file such as:

import Foundation

enum TestCase: String {
    case one = "1"
    case two = "2"
    case three = "3"

    var caseToString: String {
        switch self {
        case .one:
            return "one"
        case .two:
            return "two"
        case .three:
            return "three"
        }
    }

    var rawToString: String {
        switch rawValue {
        case "1":
            return "one"
        case "2":
            return "two"
        case "3":
            return "three"
        }
    }
}

does not change anything. With the above change to only exclude SyntaxKind.commentKinds the file gets corrected to:

import Foundation

enum TestCase: String {
    case one = "1"
    case two = "2"
    case three = "3"

    var caseToString: String {
        switch self {
        case .one:
            return "one"

        case .two:
            return "two"

        case .three:
            return "three"
        }
    }

    var rawToString: String {
        switch rawValue {
        case "1":
            return "one"

        case "2":
            return "two"

        case "3":
            return "three"
        }
    }
}

[SimplyDanny]

Would you like to open a PR to fix this, @ilendemli?

[ilendemli]

@SimplyDanny #5628