Update rubocop-rails 2.34.3 → 2.35.0 (minor)

[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-rails (2.34.3 → 2.35.0) · Repo · Changelog

Release Notes

2.35.0

Bug fixes

• #1595: Fix a false negative for Rails/I18nLocaleTexts when using redirect_back_or_to with a flash message. (@55728)
• #1587: Fix false positives for Rails/Presence with operator methods like <<, =~, and others. (@eugeneius)
• #1586: Don't add unnecessary parentheses in Rails/Presence. (@eugeneius)
• #1602: Fix an error in Rails/SelectMap when .select appears inside a subquery in an argument. (@koic)
• #1604: Allow DatabaseTypeResolvable to fall back to an adapter configuration specified in a shared key. (@codergeek121)
• #1582: Fix a false negative where local was incorrectly treated as a known environment name when using == comparison in Rails/UnknownEnv. (@lovro-bikic)

Changes

• #1571: Add more detection patterns on Rails/ResponseParsedBody. (@r7kamura)
• #1583: Extend Rails/StrongParametersExpect to detect params[:key] in method calls and raising finder methods. (@koic)
• #1584: Add support for case statements to Rails/UnknownEnv. (@lovro-bikic)
• #1592: Fix false negative for != comparison in Rails/UnknownEnv. (@lovro-bikic)
• #1598: Use glob patterns compatible with Engine or Packwerk for cops targeting spec/ and test/ directories. (@y-yagi)

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

Commits

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

• Cut 2.35.0
• Update Changelog
• Fix `KeyError` when `shared` key has no `adapter` in `database.yml`
• Merge pull request #1605 from codergeek121/support-shared-key
• Merge pull request #1603 from koic/fix_an_error_in_rails_select_map
• Fix support for shared key in database.yml
• [Fix #1602] Fix an error in `Rails/SelectMap`
• Merge pull request #1597 from 55728/fix-i18n-locale-texts-redirect-back-or-to
• [Fix #1595] Fix a false negative for `Rails/I18nLocaleTexts` when using `redirect_back_or_to`
• Merge pull request #1598 from y-yagi/globs_cops_for_test
• Use glob patterns compatible with Engine or Packwerk for cops targeting `spec/` and `test/` directories
• Merge pull request #1591 from koic/make_rails_strong_parameters_expect_allow_presence
• Merge pull request #1592 from lovro-bikic/unknown-env-inequality-comparison
• Fix false negative for != comparison in Rails/UnknownEnv
• Make `Rails/StrongParametersExpect` allow `presence`
• Suppress RuboCop's offenses
• Merge pull request #1587 from eugeneius/presence_operator_methods
• Merge pull request #1586 from eugeneius/presence_parentheses
• Don't autocorrect operator methods in Rails/Presence
• Don't add unnecessary parentheses in Rails/Presence
• Merge pull request #1583 from koic/enhance_rails_strong_parameters_expect
• Make `Rails/StrongParametersExpect` aware of `params[:key]`
• Merge pull request #1584 from lovro-bikic/rails-unknown-env-case-support
• Add support for case statements to Rails/UnknownEnv
• Merge pull request #1582 from lovro-bikic/unknown-env-local-comparison
• Fix a false negative where 'local' was incorrectly treated as a known environment name when using == comparison in Rails/UnknownEnv
• Drop `prism_spec` from the default Rake task
• Merge pull request #1571 from r7kamura/response-parsed-body-html-method
• Use RuboCop RSpec 3.9.0 for development
• Merge pull request #1564 from r7kamura/requires-gem-compatibility-remove
• Reset the docs version

✳️ rubocop (1.86.1 → 1.86.2) · Repo · Changelog

Release Notes

1.86.2

New features

• #15075: Implement true runner parallelism. (@tdeo)

Bug fixes

• #15156: Fix an error for Style/HashLookupMethod when chaining fetch (or []) calls on the same expression. (@koic)
• #15161: Fix an error for Style/ReduceToHash when nested each_with_object/inject/reduce calls would build hashes. (@koic)
• #15144: Fix an error in Style/SoleNestedConditional when autocorrecting nested conditionals containing comments. (@koic)
• #15040: Exclude constants from Style/ModuleMemberExistenceCheck. (@t-daisuke)
• #15155: Fix false negatives in Style/RedundantSelf when an explicit self receiver in one scope matches the LHS of an ||=, &&=, or op_asgn in another scope. (@koic)
• #15107: Fix false positives in Lint/RequireRelativeSelfPath when a non-.rb file uses require_relative with its own basename. (@koic)
• #15137: Fix incorrect "does not support IndentationWidth parameter" warning for Layout/ClosingParenthesisIndentation and Layout/CommentIndentation. (@koic)
• #15148: Fix false positives in Lint/RedundantSafeNavigation when safe navigation appears in rescue or ensure bodies. (@koic)
• #15147: Fix false positives in Lint/RedundantSafeNavigation when safe navigation appears in the body of unless. (@koic)
• #15163: Fix false positives in Style/Copyright when Notice pattern starts with \A#, uses \s metacharacters, or has multiple spaces after #. (@koic)
• #10179: Fix false positives in Style/DocumentDynamicEvalDefinition when the heredoc contains an escaped interpolation (\#{...}). (@eyupcanakman)
• #15154: Fix bug where specifying --out disables parallelization. (@deivid-rodriguez)
• #15106: Fix TargetFinder to work correctly inside hidden parent directories. (@alpaca-tc)
• #15102: Fix FrozenError in DisabledConfigFormatter for frozen array config parameters. (@koic)
• #15141: Fix incorrect autocorrect for Gemspec/RequireMFA causing an infinite loop when rubygems_mfa_required metadata uses a symbol key. (@koic)
• #15142: Fix infinite loop for --disable-uncorrectable and offense near heredoc. (@jonas054)
• #15054: Fix false positive for Layout/MultilineMethodCallIndentation when a dot-aligned method chain is inside a hash pair value. (@nicolas-finary)
• #15136: Fix false positive for Lint/MissingCopEnableDirective when # rubocop:disable is wrapped in a # rubocop:push / # rubocop:pop block. (@koic)
• #15115: Fix an incorrect autocorrect in Style/RegexpLiteral when the regexp contains unbalanced braces that conflict with the preferred %r delimiters. (@koic)
• #15130: Fix incorrect autocorrect in Style/Copyright when AutocorrectNotice lacks a # prefix or Notice pattern starts with ^#. (@koic)
• #14821: Fix Style/IfInsideElse autocorrect moving comments into the wrong branch when a comment precedes the nested if in an else. (@hammadxcm)
• #14583: Fix template extractor applying only the last fragment's autocorrection. (@zeronosu77108)
• #14971: Fix false positive for Style/WhileUntilModifier when the body is a conditional. (@fujitanisora)

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

Commits

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

• Cut 1.86.2
• Update Changelog
• Don't use Registry#to_h
• Simplify registry internals
• Fix false positives in `Style/Copyright`
• Fix extension suggestion documentation link
• Merge pull request #15161 from koic/fix_an_error_for_style_reduce_to_hash.md
• Merge pull request #15160 from lovro-bikic/autoload-mixins
• Autoload mixins instead of requiring them
• Fix an error for `Style/ReduceToHash` cop
• Autoload correctors instead of requiring them
• Partially revert auto-gen-config in parallel
• [Fix #15155] Fix false negatives in `Style/RedundantSelf`
• Merge pull request #15156 from koic/fix_an_error_for_style_hash_lookup_method
• Merge pull request #15154 from deivid-rodriguez/out-with-parallel
• Fix an error for `Style/HashLookupMethod` cop
• Fix bug where specifying `--out` disables parallelization
• Merge pull request #15151 from bquorning/move-exclude_limit_helper
• Merge pull request #15150 from koic/fix_false_positives_in_lint_redundant_safe_navigation_for_rescue_and_ensure
• Merge pull request #15053 from t-daisuke/dos-1st
• Merge pull request #15149 from koic/fix_false_positives_in_lint_redundant_safe_navigation_for_unless
• [Fix #15040] Exclude `constants` from `Style/ModuleMemberExistenceCheck`
• Move shared context "with exclude limit tracking"
• [Fix #15148] Fix false positives in `Lint/RedundantSafeNavigation`
• [Fix #15147] Fix false positives in `Lint/RedundantSafeNavigation`
• Merge pull request #15146 from koic/fix_an_error_in_style_sole_nested_conditional
• [Fix #15144] Fix an error for `Style/SoleNestedConditional`
• Merge pull request #15141 from koic/fix_infinite_loop_for_gemspec_require_mfa
• Merge pull request #15143 from jonas054/15142-fix-infinite-loop
• [Fix #15142] Fix unbreakable range bug for autocorrect
• Fix incorrect autocorrect for `Gemspec/RequireMFA` causing an infinite loop
• [Fix #10179] Fix `Style/DocumentDynamicEvalDefinition` false positive on escaped interpolation
• [Fix #15136] Fix false positive for `Lint/MissingCopEnableDirective` inside `# rubocop:push`/`# rubocop:pop`
• Merge pull request #15138 from koic/fix_false_positive_warning_for_two_layout_cops
• Merge pull request #15066 from nicolas-finary/fix/multiline-method-call-indentation-hash-pair
• [Fix #15137] Fix incorrect "does not support `IndentationWidth` parameter" warning
• [Fix #15054] Fix false positive for in hash pairs with block receiver
• Merge pull request #15134 from maxjacobson/mj/update-datetime-docs
• Merge pull request #14948 from hammadxcm/fix/14821-if-inside-else-comment-autocorrect
• Update Style/DateTime documentation
• Merge pull request #15132 from koic/fix_incorrect_autocorrect_in_style_copyright
• [Fix #15130] Fix incorrect autocorrect in `Style/Copyright`
• [Docs] Fix typos
• Make `InternalAffairs/LocationLineEqualityComparison` aware of `loc.begin` and `loc.end`
• Adjust range operation and add autocorrect test in `Lint/UselessAssignment`
• Merge pull request #15121 from koic/fix_false_positives_in_style_while_until_modifier
• Extend `Style/WhileUntilModifier` to skip any conditional body
• Merge pull request #15101 from sorafujitani/fix-14971-while-until-modifier-conditional-body
• Merge pull request #15116 from koic/fix_regexp_literal_autocorrect_breaks_syntax_with_curly_braces
• [Fix #15115] Fix an incorrect autocorrect in `Style/RegexpLiteral`
• Merge pull request #15112 from koic/fix_false_positive_in_lint_require_relative_self_path
• [Fix #15107] Fix false positives in `Lint/RequireRelativeSelfPath`
• Merge pull request #15110 from tdeo/true_runner_parallelism
• [Fix #15075] Implement true runner parallelism
• Merge pull request #15109 from alpaca-tc/fix-15106
• [Fix #15106] Fix TargetFinder to work correctly inside hidden parent directories
• [Fix #11232] Add --list-enabled-cops-for CLI option
• [Fix #13755] Run --auto-gen-config in parallel
• Extract read_exclude_limit spec helper for exclude_limit assertions
• Merge pull request #15103 from koic/fix_frozen-error_in_formatter_disabled_config_formatter
• Merge pull request #15104 from toy/fix-EnforcedStyleForLeadingUnderscores-header
• Fix space/typo in EnforcedStyleForLeadingUnderscores docs header
• [Fix #15102] Fix `FrozenError` in `DisabledConfigFormatter`
• [Fix #14971] Don't suggest modifier form when the body is a conditional
• Merge pull request #15100 from jonas054/11828_autocorrect_interference
• [Fix #11828] Fix brace corrector for EOL comments
• Merge pull request #14954 from zeronosu77108/fix_aggregate_template_fragment_autocorrections
• [Fix #14583] Aggregate template fragment autocorrections per iteration
• Use AST methods instead of regex to detect spaces in `Lint/ParenthesesAsGroupedExpression`
• Use AST methods instead of regex to detect empty parentheses in `Style/SymbolProc`
• Reset the antora 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)