[ruby] Update rubocop-rails → 2.21.0 (unknown)

[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.20.2 → ~> 2.21.0) · Repo · Changelog

Release Notes

2.21.0

New features

• #1075: Add new Rails/SelectMap cop that checks for uses of select(:column_name) with map(&:column_name). (@koic)
• #158: Add Rails/DangerousColumnNames cop. (@r7kamura)
• #1072: Add TransactionMethods config for Rails/TransactionExitStatement to detect custom transaction methods. (@marocchino)
• #967: Add new Rails/UnusedRenderContent cop. (@samrjenkins)
• #1052: Add explicit style to Rails/I18nLazyLookup. (@sunny)

Bug fixes

• #1078: Fix a false negative for Rails/LexicallyScopedActionFilter when no methods are defined. (@vlad-pisanov)
• #1060: Fix a false positive for Rails/HttpStatus when using symbolic value that have no numeric value mapping. (@koic)
• #1004: Fix a false-positive for Rails/RootPathnameMethods on Ruby 2.4 or lower. (@r7kamura)
• #1016: Add Rails/RedundantActiveRecordAllMethod cop. (@masato-bkn)
• #1066: Fix an error for Rails/FilePath when string interpolated Rails.root is followed by a message starting with .. (@koic)
• #1049: Fix an incorrect autocorrect for Rails/FilePath when File.join with Rails.root and path starting with /. (@ydah)
• #1045: Fix an incorrect autocorrect for Rails/NegateInclude when using Style/InverseMethods's autocorrection together. (@koic)
• #1062: Fix autocorrection for Rails/RakeEnvironment when rake task accepts arguments. (@fastjames)
• #1036: Fix an error for UniqueValidationWithoutIndex when db/schema.rb is empty. (@fatkodima)
• #1042: Fix no offences for Rails/SchemaComment when create_table with multi t columns. (@nipe0324)

Changes

• #1056: Fix database adapter detection for nested config. (@mjankowski)
• #1031: Make Lint/SafeNavigationChain allow presence_in. (@koic)
• #1080: Make Rails/HttpStatus aware of string number status. (@r7kamura)
• #1094: Make Rails/TimeZone aware of String#to_time. (@koic)
• #1015: Make Style/InvertibleUnlessCondition aware of Active Support methods. (@koic)
• #903: Read database config for Rails/BulkChangeTable from environment variable. (@joergschiller)
• #1058: Relax Include path for Rails/FindBy and Rails/FindEach. (@koic)
• #994: Restrict DuplicateAssociation cop to ActiveRecord. (@mjankowski)

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

Commits

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

• Cut 2.21.0
• Update Changelog
• Merge pull request #970 from samrjenkins/add-unused-render-content-cop
• [Fix #967] Add new Rails/UnusedRenderContent cop
• Merge pull request #1102 from koic/add_new_rails_select_map_cop
• Merge pull request #903 from joergschiller/database_from_env_bulk_change_table
• Read database config from environment variable
• Merge pull request #1101 from koic/make_rails_http_status_aware_of_string_number_status
• Merge pull request #1103 from rubocop/dependabot/github_actions/actions/checkout-4
• Bump actions/checkout from 3 to 4
• [Fix #1075] Add new `Rails/SelectMap` cop
• Merge pull request #1097 from koic/make_style_invertible_unless_condition_aware_of_as_methods
• [Fix #1080] Make `Rails/HttpStatus` aware of string number status
• Fix a build error
• Merge pull request #1099 from leonelgalan/patch-1
• Merge pull request #1100 from vlad-pisanov/vp_fix_lexically_scoped_action_filter
• [Fix #1078] Fix false negative for `Rails/LexicallyScopedActionFilter`
• Update WhereExists Documentation
• Merge pull request #1096 from koic/make_rails_time_zone_aware_of_string_to_time
• Merge pull request #1098 from oieioi/fix-comment-backquotes
• Fix incorrect comments as AsciiDoc inline code
• [Fix #1015] Make `Style/InvertibleUnlessCondition` aware of Active Support methods
• Merge pull request #1020 from masato-bkn/add_rails_redundant_all
• [Fix #1094] Make `Rails/TimeZone` aware of `String#to_time`
• Merge pull request #1088 from koic/rename_allowed_methods_to_transaction_methods
• [Fix #1016] Add `Rails/RedundantActiveRecordAllMethod` cop
• Rename `AllowedMethods` to `TransactionMethods` for `Rails/TransactionExitStatement`
• Merge pull request #1072 from marocchino/add-configurable-option-to-transaction-exit-statement
• Raise severity of `Rails/DangerousColumnNames` cop to warning
• Add Configurable option for `Rails/TransactionExitStatement`
• Rename a changelog entry file
• Merge pull request #1018 from r7kamura/dangerous-column-names
• Merge pull request #1070 from koic/fix_an_incorrect_autocorrect_for_rails_negate_include
• Merge pull request #1069 from koic/make_lint_safe_navigation_chain_allow_presence_in
• Merge pull request #1068 from koic/fix_an_error_for_rails_file_path
• [Fix #1045] Fix an incorrect autocorrect for `Rails/NegateInclude`
• [Fix #1031] Make `Lint/SafeNavigationChain` allow `presence_in`
• [Fix #1066] Fix an error for `Rails/FilePath`
• Merge pull request #1063 from fastjames/rake_environment_autocorrect_args
• [Fix #1062] Correct rake tasks with arguments
• Merge pull request #1056 from mjankowski/database-adapter-detection
• Merge pull request #1061 from koic/fix_a_false_positive_for_rails_http_status
• Merge pull request #1059 from koic/relax_include_path_for_rails_find_by_and_find_each
• [Fix #1060] Fix a false positive for `Rails/HttpStatus`
• [Fix #1058] Relax `Include` path for `Rails/FindBy` and `Rails/FindEach`
• Merge pull request #1042 from nipe0324/fix-schema-comment-with-multi-t-column
• Fix create_table with multi t columns for `Rails/SchemaComment`
• Merge pull request #1052 from sunny/add-explicit-option-to-i18n-lazy-lookup
• Fix database adapter detection for nested config
• Merge pull request #1004 from r7kamura/root-pathname-methods-ruby-version
• Add explicit option to `Rails/I18nLazyLookup`
• Merge pull request #1049 from ydah/fix-rails-file-path-autocorrect
• Fix an incorrect autocorrect for `Rails/FilePath` when File.join with Rails.root and path starting with `/`
• Merge pull request #994 from mjankowski/duplicate-associations-only-on-activerecord
• Restrict DuplicateAssociation cop to ActiveRecord
• Merge pull request #1038 from fatkodima/fix-unique_validation_without_index-empty-schema
• Fix an error for `UniqueValidationWithoutIndex` when `db/schema.rb` is empty
• Suppress new `Style/ReturnNilInPredicateMethodDefinition` 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 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)

[codecov[bot]]

Codecov Report

Merging #64 (681cd48) into main (4d988fa) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  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