[Fix #7509] Layout/SpaceInsideArrayLiteralBrackets to correct empty lines

[ayacai115]

Fixed Layout/SpaceInsideArrayLiteralBrackets to correct empty lines.
See #7509 for details.

---

Before submitting the PR make sure the following are checked:

• Wrote good commit messages.
• Commit message starts with [Fix #issue-number] (if the related issue exists).
• Feature branch is up-to-date with master (if not - rebase it).
• Squashed related commits together.
• Added tests.
• Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
• The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
• Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

[ayacai115]

@buehmann Thanks for the suggestion!
master...buehmann:empty-array-literal/7509

[Drenmi]

Normally, the edge case that comes up for these kinds of cops is with comments, e.g.:

foo = [
# Intentionally left empty
]

[ayacai115]

@Drenmi
That's true.

What I'd like to fix is that rubocop raises a warning and says "corrected" but it's not actually corrected, which is a bug.

Regarding your suggestion, it's about how the syntax rule should be, so maybe we can discuss in another issue?

[Drenmi]

What I'd like to fix is that rubocop raises a warning and says "corrected" but it's not actually corrected, which is a bug.

I see. 🙂 Yes. My comment is a bit tangential then, although fixing the bug is likely going to cause comments to be destroyed on auto-correct, so I think we should at least open a new issue for it, if it's not in scope of this PR. 👍

[ayacai115]

@Drenmi Thanks for your suggestion😄
I opened a issue for that: #7512

[buehmann]

Normally, the edge case that comes up for these kinds of cops is with comments, e.g.:

That's not an issue here because the "empty brackets" detection works on the token level.

[koic]

Thanks!