rubocops/cask/array_alphabetization: skip blank lines when sorting

[krehel]

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same change?
• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes? Here's an example.
• Have you successfully run brew style with your changes locally?
• Have you successfully run brew typecheck with your changes locally?
• Have you successfully run brew tests with your changes locally?

---

Minor fix to sorting lines when alphabetizing arrays, as empty lines (or all whitespace) cause an apparent break. This simply skips those when rewriting zap and uininstall stanzas.

Cop Layout/EmptyLines will correct multiple blank lines in a row, but something like the below won't be fixed (example with the expressvpn Cask)

  zap trash: [
    "/Library/Application Support/com.expressvpn.ExpressVPN",
    "/Library/LaunchDaemons/com.expressvpn.expressvpnd.plist",
    "~/Library/Application Support/com.expressvpn.ExpressVPN",
    "~/Library/HTTPStorages/com.expressvpn.ExpressVPN",

    "~/Library/Logs/ExpressVPN",
    "~/Library/Preferences/com.expressvpn.ExpressVPN.plist",
  ]

Which results in An error occurred while Cask/ArrayAlphabetization cop was inspecting /opt/homebrew/Library/Taps/homebrew/homebrew-cask/Casks/e/expressvpn.rb:25:2.

[issyl0]

This is a personal preference thing but I think I'd prefer a separate next for this rather than an ||.

[MikeMcQuaid]

Looks good with @issyl0's suggestion applied!

[krehel]

No worries! I did it that way first, and then decided I'd || it. I'll apply this suggestion!

[issyl0]

Thanks Justin!