Skip to content

dep: bump rubocop-rails from 2.16.1 to 2.17.0

George Koltsov requested to merge dependabot-bundler-rubocop-rails-2.17.0 into main

Bumps rubocop-rails from 2.16.1 to 2.17.0.

Release notes

Sourced from rubocop-rails's releases.

RuboCop Rails 2.17.0

New features

Bug fixes

  • #786: Fix a false negative for Rails/ActionControllerTestCase when extending ActionController::TestCase and having a method definition. (@​koic)
  • #792: Fix a false negative for Rails/RedundantPresenceValidationOnBelongsTo when belongs_to at least one block and one hash like belongs_to :company, -> { where(foo: true) }, inverse_of: :employee. (@​PedroAugustoRamalhoDuarte)
  • #781: Make Rails/DynamicFindBy aware of find_by_token_for. (@​koic)
  • #809: Fix an error for Rails/FreezeTime when using travel_to without argument. (@​koic)
  • #794: Fix an error for Rails/RedundantReceiverInWithOptions when calling a method with a receiver in with_options without block arguments. (@​koic)
  • #782: Fix an incorrect autocorrect for Rails/EagerEvaluationLogMessage when using Style/MethodCallWithArgsParentheses's autocorrection together. (@​koic)
  • #776: Fix an incorrect autocorrect for Rails/Presence when using arithmetic operation in else branch. (@​koic)
  • #813: Fix errors that occur when unrelated tag is investigated by Rails/ContentTag. (@​r7kamura)
  • #808: Fix false positive for Rails/ActionControllerFlashBeforeRender when render call precedes flash call. (@​americodls)
  • #778: Fix a false positive for Rails/DynamicFindBy when using page.find_by_id as a Capybara testing API. (@​koic)
  • #816: Fix an incorrect autocorrect for Rails/Presence when a right-hand side of the relational operator. (@​ydah)

Changes

Changelog

Sourced from rubocop-rails's changelog.

2.17.0 (2022-10-22)

New features

Bug fixes

  • #786: Fix a false negative for Rails/ActionControllerTestCase when extending ActionController::TestCase and having a method definition. ([@​koic][])
  • #792: Fix a false negative for Rails/RedundantPresenceValidationOnBelongsTo when belongs_to at least one block and one hash like belongs_to :company, -> { where(foo: true) }, inverse_of: :employee. ([@​PedroAugustoRamalhoDuarte][])
  • #781: Make Rails/DynamicFindBy aware of find_by_token_for. ([@​koic][])
  • #809: Fix an error for Rails/FreezeTime when using travel_to without argument. ([@​koic][])
  • #794: Fix an error for Rails/RedundantReceiverInWithOptions when calling a method with a receiver in with_options without block arguments. ([@​koic][])
  • #782: Fix an incorrect autocorrect for Rails/EagerEvaluationLogMessage when using Style/MethodCallWithArgsParentheses's autocorrection together. ([@​koic][])
  • #776: Fix an incorrect autocorrect for Rails/Presence when using arithmetic operation in else branch. ([@​koic][])
  • #813: Fix errors that occur when unrelated tag is investigated by Rails/ContentTag. ([@​r7kamura][])
  • #808: Fix false positive for Rails/ActionControllerFlashBeforeRender when render call precedes flash call. ([@​americodls][])
  • #778: Fix a false positive for Rails/DynamicFindBy when using page.find_by_id as a Capybara testing API. ([@​koic][])
  • #816: Fix an incorrect autocorrect for Rails/Presence when a right-hand side of the relational operator. ([@​ydah][])

Changes

  • #779: Add mail to AllowedMethods of Style/SymbolProc. ([@​koic][])
  • #796: Add several directories to Exclude to prevent slow investigation. ([@​koic][])
  • #822: Extends Rails/HttpStatus cop to check routes.rb. ([@​anthony-robin][])
  • #787: Make Rails/Pluck aware of all keys. ([@​koic][])
  • #800: Make Rails/TimeZone aware of timezone UTF offset. ([@​inkstak][])
Commits
  • 177c92f Cut 2.17.0
  • 1cb96f9 Update Changelog
  • e40d8b9 Merge pull request #823 from anthony-robin/http-status-routes
  • 1dcbda1 [Fix #822] Extends Rails/HttpStatus cop to check routes.rb
  • 6051ed8 Merge pull request #806 from koic/add_mail_to_allowed_methods_of_style_symbol...
  • 25d9dfb Merge pull request #816 from ydah/fix/presence
  • 4b6362e Fix an incorrect autocorrect for Rails/Presence when a right-hand side of t...
  • 7a434e4 Tweak an changelog for Rails/SetIgnoredColumns
  • d7db4d1 Merge pull request #817 from ydah/change/presence
  • 43fa456 Preserve multiline semantics on Rails/Presence
  • Additional commits viewable in compare view

Dependabot commands
You can trigger Dependabot actions by commenting on this MR
  • @dependabot-bot rebase will rebase this MR
  • @dependabot-bot recreate will recreate this MR rewriting all the manual changes and resolving conflicts

Allure report

allure-report-publisher generated test report!

rspec: ✅ test report for 48198c61

+------------------------------------------------------------------+
|                        behaviors summary                         |
+-------------+--------+--------+---------+-------+-------+--------+
|             | passed | failed | skipped | flaky | total | result |
+-------------+--------+--------+---------+-------+-------+--------+
| services    | 180    | 0      | 0       | 0     | 180   | ✅     |
| controllers | 25     | 0      | 0       | 0     | 25    | ✅     |
| models      | 4      | 0      | 0       | 0     | 4     | ✅     |
| jobs        | 14     | 0      | 0       | 0     | 14    | ✅     |
| tasks       | 9      | 0      | 0       | 0     | 9     | ✅     |
| system      | 10     | 0      | 0       | 0     | 10    | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+
| Total       | 242    | 0      | 0       | 0     | 242   | ✅     |
+-------------+--------+--------+---------+-------+-------+--------+
Edited by George Koltsov

Merge request reports