Bump rubocop from 1.68.0 to 1.69.0

[dependabot]

Bumps rubocop from 1.68.0 to 1.69.0.

Release notes

Sourced from rubocop's releases.

RuboCop 1.69

New features

• #13439: Add new Lint/HashNewWithKeywordArgumentsAsDefault cop. (@​koic)
• #11191: Add new Lint/NumericOperationWithConstantResult cop. ([@​zopolis4][])
• #13486: Add new Style/DigChain cop. ([@​dvandersluis][])
• #13490: Add new Style/FileNull cop. ([@​dvandersluis][])
• #13484: Add new Style/FileTouch cop. ([@​lovro-bikic][])
• #13437: Add a new cop Lint/UselessDefined to detect cases such as defined?('Foo') when defined?(Foo) was intended. ([@​earlopain][])

Bug fixes

• #13455: Fix a false positive for Layout/EmptyLineAfterGuardClause when using a guard clause outside oneliner block. (@​koic)
• #13412: Fix a false positive for Style/RedundantLineContinuation when there is a line continuation at the end of Ruby code followed by __END__ data. (@​koic)
• #13476: Allow to write generics type of RBS::Inline annotation after subclass definition in Style/CommentedKeyword. ([@​dak2][])
• #13441: Fix an incorrect autocorrect for Style/IfWithSemicolon when using return with value in if with a semicolon is used. (@​koic)
• #13448: Fix an incorrect autocorrect for Style/IfWithSemicolon when the then body contains an arithmetic operator method call with an argument. (@​koic)
• #13199: Make Style/RedundantCondition skip autocorrection when a branch has a comment. (@​koic)
• #13411: Fix Style/BitwisePredicate when having regular method. ([@​d4be4st][])
• #13432: Fix false positive for Lint/FloatComparison against nil. ([@​lovro-bikic][])
• #13461: Fix false positives for Lint/InterpolationCheck when using invalid syntax in interpolation. (@​koic)
• #13402: Fix a false positive for Lint/SafeNavigationConsistency when using unsafe navigation with both && and ||. (@​koic)
• #13434: Fix a false positive for Naming/MemoizedInstanceVariableName for assignment methods`. ([@​earlopain][])
• #13415: Fix false positives for Naming/MemoizedInstanceVariableName when using initialize_clone, initialize_copy, or initialize_dup. (@​koic)
• #13421: Fix false positives for Style/SafeNavigation when using a method chain that exceeds the MaxChainLength value and includes safe navigation operator. (@​koic)
• #13433: Fix autocorrection for Style/AccessModifierDeclarations for multiple inline symbols. ([@​dvandersluis][])
• #13430: Fix EmptyLinesAroundMethodBody for methods with arguments spanning multiple lines. ([@​aduth][])
• #13438: Fix incorrect correction in Lint/Void if an operator is called in a void context using a dot. ([@​dvandersluis][])
• #13419: Fix Lint/DeprecatedOpenSSLConstant false positive when the argument is a safe navigation method call. ([@​dvandersluis][])
• #13404: Fix Style/AccessModifierDeclarations to register (as positive or negative, depending on AllowModifiersOnSymbols value) access modifiers with multiple symbols. ([@​dvandersluis][])
• #13436: Fix incorrect offense and autocorrect for Lint/RedundantSplatExpansion when percent literal array is used in a safe navigation method call. ([@​lovro-bikic][])
• #13442: Fix an incorrect autocorrect for Style/NestedTernaryOperator when ternary operators are nested and the inner condition is parenthesized. (@​koic)
• #13444: Fix an incorrect autocorrect for Style/OneLineConditional when the else branch of a ternary operator has multiple expressions. (@​koic)
• #13483: Fix an incorrect autocorrect for Style/RedundantRegexpArgument when using escaped double quote character. (@​koic)
• #13497: Fix infinite loop error for Style/IfWithSemicolon when using nested if/;/end in if body. (@​koic)
• #13477: Update Layout/LeadingCommentSpace to accept multiline shebangs at the top of the file. ([@​dvandersluis][])
• #13453: Update Style/AccessModifierDeclarations to handle attr_* methods with multiple parameters. ([@​dvandersluis][])
• #12597: Update Style/SingleLineDoEndBlock to not register an offense if it will introduce a conflicting Layout/RedundantLineBreak offense. ([@​dvandersluis][])

Changes

• #11680: Add autocorrection for strings to Layout/LineLength when SplitStrings is set to true. ([@​dvandersluis][])
• #13470: Make Style/ArrayIntersect aware of none?. ([@​earlopain][])
• #13481: Support unicode-display_width v3. ([@​gemmaro][])
• #13473: Update Lint/ItWithoutArgumentsInBlock to not register offenses in Ruby 3.4. ([@​dvandersluis][])
• #13420: Update Lint/RedundantSafeNavigation to register an offense when the receiver is self. ([@​dvandersluis][])
• #11393: Update Lint/UnusedMethodArgument to allow the class names for IgnoreNotImplementedMethods to be configured. ([@​dvandersluis][])
• #13058: Update Style/AccessModifierDeclarations to accept modifier with splatted method call. ([@​dvandersluis][])

... (truncated)

Changelog

Sourced from rubocop's changelog.

1.69.0 (2024-11-26)

New features

• #13439: Add new Lint/HashNewWithKeywordArgumentsAsDefault cop. ([@​koic][])
• #11191: Add new Lint/NumericOperationWithConstantResult cop. ([@​zopolis4][])
• #13486: Add new Style/DigChain cop. ([@​dvandersluis][])
• #13490: Add new Style/FileNull cop. ([@​dvandersluis][])
• #13484: Add new Style/FileTouch cop. ([@​lovro-bikic][])
• #13437: Add a new cop Lint/UselessDefined to detect cases such as defined?('Foo') when defined?(Foo) was intended. ([@​earlopain][])

Bug fixes

• #13455: Fix a false positive for Layout/EmptyLineAfterGuardClause when using a guard clause outside oneliner block. ([@​koic][])
• #13412: Fix a false positive for Style/RedundantLineContinuation when there is a line continuation at the end of Ruby code followed by __END__ data. ([@​koic][])
• #13476: Allow to write generics type of RBS::Inline annotation after subclass definition in Style/CommentedKeyword. ([@​dak2][])
• #13441: Fix an incorrect autocorrect for Style/IfWithSemicolon when using return with value in if with a semicolon is used. ([@​koic][])
• #13448: Fix an incorrect autocorrect for Style/IfWithSemicolon when the then body contains an arithmetic operator method call with an argument. ([@​koic][])
• #13199: Make Style/RedundantCondition skip autocorrection when a branch has a comment. ([@​koic][])
• #13411: Fix Style/BitwisePredicate when having regular method. ([@​d4be4st][])
• #13432: Fix false positive for Lint/FloatComparison against nil. ([@​lovro-bikic][])
• #13461: Fix false positives for Lint/InterpolationCheck when using invalid syntax in interpolation. ([@​koic][])
• #13402: Fix a false positive for Lint/SafeNavigationConsistency when using unsafe navigation with both && and ||. ([@​koic][])
• #13434: Fix a false positive for Naming/MemoizedInstanceVariableName for assignment methods`. ([@​earlopain][])
• #13415: Fix false positives for Naming/MemoizedInstanceVariableName when using initialize_clone, initialize_copy, or initialize_dup. ([@​koic][])
• #13421: Fix false positives for Style/SafeNavigation when using a method chain that exceeds the MaxChainLength value and includes safe navigation operator. ([@​koic][])
• #13433: Fix autocorrection for Style/AccessModifierDeclarations for multiple inline symbols. ([@​dvandersluis][])
• #13430: Fix EmptyLinesAroundMethodBody for methods with arguments spanning multiple lines. ([@​aduth][])
• #13438: Fix incorrect correction in Lint/Void if an operator is called in a void context using a dot. ([@​dvandersluis][])
• #13419: Fix Lint/DeprecatedOpenSSLConstant false positive when the argument is a safe navigation method call. ([@​dvandersluis][])
• #13404: Fix Style/AccessModifierDeclarations to register (as positive or negative, depending on AllowModifiersOnSymbols value) access modifiers with multiple symbols. ([@​dvandersluis][])
• #13436: Fix incorrect offense and autocorrect for Lint/RedundantSplatExpansion when percent literal array is used in a safe navigation method call. ([@​lovro-bikic][])
• #13442: Fix an incorrect autocorrect for Style/NestedTernaryOperator when ternary operators are nested and the inner condition is parenthesized. ([@​koic][])
• #13444: Fix an incorrect autocorrect for Style/OneLineConditional when the else branch of a ternary operator has multiple expressions. ([@​koic][])
• #13483: Fix an incorrect autocorrect for Style/RedundantRegexpArgument when using escaped double quote character. ([@​koic][])
• #13497: Fix infinite loop error for Style/IfWithSemicolon when using nested if/;/end in if body. ([@​koic][])
• #13477: Update Layout/LeadingCommentSpace to accept multiline shebangs at the top of the file. ([@​dvandersluis][])
• #13453: Update Style/AccessModifierDeclarations to handle attr_* methods with multiple parameters. ([@​dvandersluis][])
• #12597: Update Style/SingleLineDoEndBlock to not register an offense if it will introduce a conflicting Layout/RedundantLineBreak offense. ([@​dvandersluis][])

Changes

• #11680: Add autocorrection for strings to Layout/LineLength when SplitStrings is set to true. ([@​dvandersluis][])
• #13470: Make Style/ArrayIntersect aware of none?. ([@​earlopain][])
• #13481: Support unicode-display_width v3. ([@​gemmaro][])
• #13473: Update Lint/ItWithoutArgumentsInBlock to not register offenses in Ruby 3.4. ([@​dvandersluis][])
• #13420: Update Lint/RedundantSafeNavigation to register an offense when the receiver is self. ([@​dvandersluis][])
• #11393: Update Lint/UnusedMethodArgument to allow the class names for IgnoreNotImplementedMethods to be configured. ([@​dvandersluis][])
• #13058: Update Style/AccessModifierDeclarations to accept modifier with splatted method call. ([@​dvandersluis][])

Commits

• 9a49a33 Cut 1.69
• 96833cd Update Changelog
• 32c8dd5 Merge pull request #13498 from dvandersluis/useless-include-util
• 2904985 Remove unnecessary include Util
• 77f3e6f Create Style/FileTouch cop
• 8ac8947 Fix infinite loop error for Style/IfWithSemicolon
• beb22bc Update Style/SingleArgumentDig to not register offenses on chained dig ca...
• c8cdb3d [Fix #13486] Add new Style/DigChain cop
• efc3fd8 [Fix #13490] Add new Style/FileNull cop
• 8717f11 Remove unnecessary include IgnoredNode
• Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot merge will merge this PR after your CI passes on it
• @dependabot squash and merge will squash and merge this PR after your CI passes on it
• @dependabot cancel merge will cancel a previously requested merge and block automerging
• @dependabot reopen will reopen this PR if it is closed
• @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
• @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

[github-actions]

Backend-review-group approval confirmed.

[rjohnson2011]

Can someone from @department-of-veterans-affairs/lighthouse-dash or @department-of-veterans-affairs/octo-identity please approve?