[ruby] Upgrade rubocop-rails to version 2.19.0

[depfu[bot]]

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

What changed?

✳️ rubocop-rails (~> 2.18.0 → ~> 2.19.0) · Repo · Changelog

Release Notes

2.19.0

New features

• #337: Add new Rails/ThreeStateBooleanColumn cop. (@fatkodima)
• #45: Make Rails/Delegate aware of self. (@koic)

Bug fixes

• #934: Fix a false negative for Rails/Output when print methods without arguments. (@koic)
• #99: Fix a false positive for Rails/HttpPositionalArguments when using include Rack::Test::Methods. (@koic)
• #501: Fix a false positive for Rails/OutputSafety when using html_safe for I18n methods. (@koic)
• #860: Fix a false positive for Rails/Pluck when using regexp literal key for String#[]. (@koic)
• #938: Fix an error for Rails/WhereNotWithMultipleConditions when using where.not.lt(condition) as a Mongoid API'. (@koic)
• #941: Remove redundant config for Style/InverseMethods. (@koic)

Changes

• #299: Add autocorrection for Rails/ReflectionClassName. (@tejasbubane)
• #34: Allow CreateTableWithTimestamps when using id: false and not include timestamps. (@koic)
• #694: Mark Rails/FindEach as unsafe. (@koic)
• #940: Mark Rails/ResponseParsedBody as unsafe. (@koic)

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

Commits

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

• Cut 2.19.0
• Update Changelog
• Merge pull request #976 from koic/fix_a_false_positive_for_rails_output_safety
• Merge pull request #975 from koic/allow_create_table_with_timestamps_for_id_false
• Merge pull request #974 from koic/make_rails_delegate_aware_of_self
• Use `each_comment_in_lines` instead of deprecated API
• Remove `Style/InverseMethods` config for `exists?`
• Merge pull request #973 from koic/make_style_inverse_methods_aware_of_exists_predicate
• Merge pull request #972 from koic/mark_rails_find_each_as_unsafe
• Merge pull request #971 from koic/mark_rails_response_parsed_body_as_unsafe
• [Fix #501] Fix a false positive for `Rails/OutputSafety`
• [Fix #34] Allow `CreateTableWithTimestamps` when using `id: false` and not include `timestamps`
• [Fix #45] Make `Rails/Delegate` aware of `self`
• [Fix #888] Make `Style/InverseMethods` aware of `exists?`
• [Fix #694] Mark `Rails/FindEach` as unsafe
• Merge pull request #299 from tejasbubane/reflection-class-name-autocorrect
• [Fix #940] Mark `Rails/ResponseParsedBody` as unsafe
• Add autocorrection for `Rails/ReflectionClassName` cop
• Remove redundant `to_s` suggestion from offense message for `Rails/FilePath`
• Merge pull request #969 from koic/fix_a_false_positive_for_rails_http_positional_arguments
• Merge pull request #968 from koic/fix_incompatible_suggestion_mesage_for_rails_file_path
• [Fix #99] Fix a false positive for `Rails/HttpPositionalArguments`
• [Fix #472] Fix incompatible suggestion message for `Rails/FilePath`
• Merge pull request #676 from fatkodima/three-state-boolean
• Merge pull request #965 from koic/fix_a_false_negative_for_rails_output
• Merge pull request #964 from koic/remove_redundant_config_for_style_inverse_methods
• Merge pull request #963 from koic/fix_a_false_positive_for_rails_pluck
• [Fix #934] Fix a false negative for `Rails/Output`
• [Fix #941] Remove redundant config for `Style/InverseMethods`
• [Fix #860] Fix a false positive for `Rails/Pluck`
• Merge pull request #961 from tmaier/patch-1
• Do not fail rubocop when using in generator
• Suppress new `Style/MapToSet` offense
• Merge pull request #956 from koic/add_rails_tips_to_docs
• Merge pull request #957 from koic/sync_spell_checking_workflow
• Use Codespell GitHub Actions
• Suppress new `InternalAffairs/RedundantSourceRange` offenses
• [Doc] Add `config.generators.after_generate` Rails config to docs
• Merge pull request #943 from fatkodima/require-missing-blank
• Merge pull request #945 from ybiquitous/patch-1
• Fix typo in `Rails/UniqueValidationWithoutIndex` doc
• Merge pull request #954 from koic/fix_an_error_for_rails_where_not_with_multiple_conditions
• [Fix #938] Fix an error for `Rails/WhereNotWithMultipleConditions`
• [Doc] Add `$` to console commands
• Fix typos
• Require missing `#blank?` from ActiveSupport
• Add new `Rails/ThreeStateBoolean` cop
• Suppress `InternalAffairs/RedundantSourceRange` offense
• [Fix #939] Remove missing references for `Rails/ActionControllerFlashBeforeRender`
• Fix an error when using RuboCop 1.46.0 or lower
• Suppress `InternalAffairs/LocationExpression` offenses
• Switch back docs version to master

---

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 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)

[codecov[bot]]

Codecov Report

Merging #54 (4c5c97a) into main (d98e256) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #54   +/-   ##
=======================================
  Coverage   98.36%   98.36%           
=======================================
  Files           8        8           
  Lines         244      244           
=======================================
  Hits          240      240           
  Misses          4        4           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more