Skip to content

dep: bump rubocop-rails from 2.17.2 to 2.17.3

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

Bumps rubocop-rails from 2.17.2 to 2.17.3.

Release notes

Sourced from rubocop-rails's releases.

RuboCop Rails 2.17.3

% cat relnotes/v2.17.3.md

Bug fixes

  • #843: Fix a false positive for Rails/ActionControllerFlashBeforeRender when using flash in multiline if branch before redirect_to. (@​koic)
  • #841: Fix an error for Rails/ActionOrder when using unconventional order of multiple actions. (@​koic)
  • #838: Fix an incorrect autocorrect for Rails/ActionOrder when using unconventional order of actions in conditions. (@​koic)
  • #867: Fix autocorrection bug when ::Hash is used on Rails/IndexBy and Rails/IndexWith. (@​r7kamura)
  • #848: Fix a false positive for Rails/FreezeTime when using travel_to with an argument of Time.new(...).in_time_zone. (@​koic)
  • #869: Fix false-positives that non Rails formats are offended on Rails/ToSWithArgument. (@​r7kamura)
  • #866: Fix false-positives when constant is used with receiver on Rails/DurationArithmetic, Rails/IndexBy, Rails/IndexWIth, and Rails/RequireDependency. (@​r7kamura)
  • #850: Fix default configuration for Rails/I18nLazyLookup. (@​vlad-pisanov)
  • #837: Fix incorrect autocorrection of Rails/ActionOrder about comments. (@​r7kamura)
  • #855: Fix Rails/RootPathnameMethods autocorrection for Pathname calls without parens. (@​gsamokovarov)
  • #868: Support :: prefixed constants on Rails/ActionControllerFlashBeforeRender, Rails/ActionControllerTestCase, Rails/ApplicationController, Rails/ApplicationJob, Rails/ApplicationMailer, Rails/ApplicationRecord, Rails/DotSeparatedKeys, Rails/DynamicFindBy, Rails/FindEach, Rails/FreezeTime, Rails/HasManyOrHasOneDependent, Rails/HelperInstanceVariable, Rails/MailerName, Rails/MigrationClassName, Rails/Output, Rails/ReversibleMigrationMethodDefinition, Rails/ReversibleMigration, Rails/ShortI18n, Rails/SkipsModelValidations, and Rails/TimeZoneAssignment. (@​r7kamura)
  • #865: Support ::Rails and ::File on Rails/FilePath cop. (@​r7kamura)

Changes

  • #862: Ignore if number of arguments does not match on Rails/DynamicFindBy. (@​r7kamura)
Changelog

Sourced from rubocop-rails's changelog.

2.17.3 (2022-11-20)

Bug fixes

  • #843: Fix a false positive for Rails/ActionControllerFlashBeforeRender when using flash in multiline if branch before redirect_to. ([@​koic][])
  • #841: Fix an error for Rails/ActionOrder when using unconventional order of multiple actions. ([@​koic][])
  • #838: Fix an incorrect autocorrect for Rails/ActionOrder when using unconventional order of actions in conditions. ([@​koic][])
  • #867: Fix autocorrection bug when ::Hash is used on Rails/IndexBy and Rails/IndexWith. ([@​r7kamura][])
  • #848: Fix a false positive for Rails/FreezeTime when using travel_to with an argument of Time.new(...).in_time_zone. ([@​koic][])
  • #869: Fix false-positives that non Rails formats are offended on Rails/ToSWithArgument. ([@​r7kamura][])
  • #866: Fix false-positives when constant is used with receiver on Rails/DurationArithmetic, Rails/IndexBy, Rails/IndexWIth, and Rails/RequireDependency. ([@​r7kamura][])
  • #850: Fix default configuration for Rails/I18nLazyLookup. ([@​vlad-pisanov][])
  • #837: Fix incorrect autocorrection of Rails/ActionOrder about comments. ([@​r7kamura][])
  • #855: Fix Rails/RootPathnameMethods autocorrection for Pathname calls without parens. ([@​gsamokovarov][])
  • #868: Support :: prefixed constants on Rails/ActionControllerFlashBeforeRender, Rails/ActionControllerTestCase, Rails/ApplicationController, Rails/ApplicationJob, Rails/ApplicationMailer, Rails/ApplicationRecord, Rails/DotSeparatedKeys, Rails/DynamicFindBy, Rails/FindEach, Rails/FreezeTime, Rails/HasManyOrHasOneDependent, Rails/HelperInstanceVariable, Rails/MailerName, Rails/MigrationClassName, Rails/Output, Rails/ReversibleMigrationMethodDefinition, Rails/ReversibleMigration, Rails/ShortI18n, Rails/SkipsModelValidations, and Rails/TimeZoneAssignment. ([@​r7kamura][])
  • #865: Support ::Rails and ::File on Rails/FilePath cop. ([@​r7kamura][])

Changes

  • #862: Ignore if number of arguments does not match on Rails/DynamicFindBy. ([@​r7kamura][])
Commits
  • e4f748d Cut 2.17.3
  • 3865328 Update Changelog
  • 2a07001 Merge pull request #868 from r7kamura/feature/add-cbase-support
  • 43acab1 Merge pull request #869 from r7kamura/feature/to-s-false
  • f9a3200 Fix false-positives that non Rails formats are offended on `Rails/ToSWithArgu...
  • c680d24 Merge pull request #866 from r7kamura/feature/fix-const-nil-cbase
  • 3191081 Fix false-positives when constant is used with receiver
  • 3d7228e Merge pull request #867 from r7kamura/feature/index-method
  • 20042f6 Support :: prefixed constants on some cops
  • 53bf6ab Fix autocorrection bug when ::Hash is used on Rails/IndexBy and `Rails/In...
  • 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 6a76fc17

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

Merge request reports