Update rubocop 1.56.2 → 1.56.3 (patch)

[depfu]

Here is everything you need to know about this update. Please take a good look at what changed and the test results before merging this pull request.

What changed?

✳️ rubocop (1.56.2 → 1.56.3) · Repo · Changelog

Release Notes

1.56.3

Bug fixes

• #12151: Make Layout/EmptyLineAfterGuardClause allow :nocov: directive after guard clause. (@koic)
• #12195: Fix a false negative for Layout/SpaceAfterNot when a newline is present after !. (@ymap)
• #12192: Fix a false positive for Layout/RedundantLineBreak when using quoted symbols with a single newline. (@ymap)
• #12190: Fix a false positive for Layout/SpaceAroundOperators when aligning operators vertically. (@koic)
• #12171: Fix a false positive for Style/ArrayIntersect when using block argument for Enumerable#any?. (@koic)
• #12172: Fix a false positive for Style/EmptyCaseCondition when using return, break, next or method call before empty case condition. (@koic)
• #12162: Fix an error for Bundler/DuplicatedGroup when there's a duplicate set of groups and the group value contains a splat. (@koic)
• #12182: Fix an error for Lint/UselessAssignment when variables are assigned using chained assignment and remain unreferenced. (@koic)
• #12181: Fix an incorrect autocorrect for Lint/UselessAssignment when variables are assigned with sequential assignment using the comma operator and unreferenced. (@koic)
• #12187: Fix an incorrect autocorrect for Style/SoleNestedConditional when comment is in an empty nested if body. (@ymap)
• #12183: Fix an incorrect autocorrect for Style/MultilineTernaryOperator when returning a multiline ternary operator expression with safe navigation method call. (@koic)
• #12168: Fix bug in Style/ArgumentsForwarding when there are repeated send nodes. (@owst)
• #12185: Set target version for Layout/HeredocIndentation. (@tagliala)

Does any of this look wrong? Please let us know.

Commits

See the full diff on Github. The new version differs by 36 commits:

• Cut 1.56.3
• Move a changelog entry
• Update Changelog
• Try avoiding project_root leak to fix flaky error
• Fix isolated test files should not break isolation.
• Fix shadowing of FileFinder stop_dir in isolated tests.
• [Fix #12151] Make `Layout/EmptyLineAfterGuardClause` allow `:nocov:` directive
• Constantize fixed arrays of `Style/SymbolArray`
• Use RuboCop RSpec 2.24
• [Fix #12195] Fix a false negative for `Layout/SpaceAfterNot`
• [Fix #12181] Fix an incorrect autocorrect for `Lint/UselessAssignment`
• [Fix #12171] Fix a false positive for `Style/ArrayIntersect`
• [Docs] Tweak the doc for `Style/YodaExpression`
• [Fix #12182] Fix an error for `Lint/UselessAssignment`
• Merge pull request #12193 from ymap/fix_a_false_positive_for_layout_redundant_line_break
• [Fix #12192] Fix a false positive for `Layout/RedundantLineBreak`
• Merge pull request #12191 from koic/fix_a_false_positive_for_layout_space_around_operators
• Merge pull request #12185 from tagliala/bugfix/add-minimum-required-ruby-version-to-layout-heredoc-indentation
• [Fix #12190] Fix a false positive for `Layout/SpaceAroundOperators`
• Set target version for `Layout/HeredocIndentation`
• Merge pull request #12188 from ymap/fix_an_incorrect_autocorrect_for_style_sole_nested_conditional
• Merge pull request #12189 from rubocop/dependabot/github_actions/actions/checkout-4
• Bump actions/checkout from 3 to 4
• [Fix rubocop#12187] Fix an incorrect autocorrect for `Style/SoleNestedConditional`
• Remove comments in `Style/InvertibleUnlessCondition` for Rails
• Merge pull request #12183 from koic/fix_an_incorrrect_autocorrect_for_style_multiline_ternary_operator
• Fix an incorrect autocorrect for `Style/MultilineTernaryOperator`
• [Docs] Add Helix to the LSP doc
• Merge pull request #12174 from koic/fix_a_false_positive_for_style_empty_case_condition
• Merge pull request #12169 from owst/fix_bug_in_style_arguments_forwarding_with_duplicate_sends
• [Fix #12168]: Fix bug in `Style/ArgumentsForwarding`
• [Fix #12172] Fix a false positive for `Style/EmptyCaseCondition`
• Merge pull request #12170 from koic/make_internal_affairs_redundant_method_dispatch_node_aware_of_method_p
• Make `InternalAffairs/RedundantMethodDispatchNode` aware of `method?`
• [Fix #12162] Fix an error for `Bundler/DuplicatedGroup`
• Switch back the docs version

---

Depfu will automatically keep this PR conflict-free, as long as you don't add any commits to this branch yourself. You can also trigger a rebase manually by commenting with @depfu rebase.

All Depfu comment commands

@​depfu rebase

Rebases against your default branch and redoes this update

@​depfu recreate

Recreates this PR, overwriting any edits that you've made to it

@​depfu merge

Merges this PR once your tests are passing and conflicts are resolved

@​depfu cancel merge

Cancels automatic merging of this PR

@​depfu close

Closes this PR and deletes the branch

@​depfu reopen

Restores the branch and reopens this PR (if it's closed)

@​depfu pause

Ignores all future updates for this dependency and closes this PR

@​depfu pause [minor|major]

Ignores all future minor/major updates for this dependency and closes this PR

@​depfu resume

Future versions of this dependency will create PRs again (leaves this PR as is)

[depfu]

Closing because this update has already been applied