KEMBAR78
[Fix #14362] Update `Lint/RequireRangeParentheses` to not register false positives when range elements span multiple lines by dvandersluis · Pull Request #14363 · rubocop/rubocop · GitHub
Skip to content

Conversation

@dvandersluis
Copy link
Member

Update Lint/RequireRangeParentheses to not register a false positive when the begin element spans multiple lines. If the operator is on the same line as the end element, there is no ambiguity.

Fixes #14362.


Before submitting the PR make sure the following are checked:

  • The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
  • 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.
  • Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
  • Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

…ster false positives when range elements span multiple lines
@koic koic merged commit 8739ea0 into rubocop:master Jul 14, 2025
24 checks passed
@dvandersluis dvandersluis deleted the issue/14362 branch July 14, 2025 16:03
@marcofognog
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

False positive registering of offense by Lint/RequireRangeParentheses when Date range has line break (non endless range)

3 participants