[src/api] Update rubocop-rails: 2.17.2 → 2.17.3 (patch)

[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.17.2 → 2.17.3) · Repo · Changelog

Release Notes

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)

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

Commits

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

• Cut 2.17.3
• Update Changelog
• Merge pull request #868 from r7kamura/feature/add-cbase-support
• Merge pull request #869 from r7kamura/feature/to-s-false
• Fix false-positives that non Rails formats are offended on `Rails/ToSWithArgument`
• Merge pull request #866 from r7kamura/feature/fix-const-nil-cbase
• Fix false-positives when constant is used with receiver
• Merge pull request #867 from r7kamura/feature/index-method
• Support `::` prefixed constants on some cops
• Fix autocorrection bug when `::Hash` is used on `Rails/IndexBy` and `Rails/IndexWith`
• Merge pull request #865 from r7kamura/feature/file-path-cbase
• Support `::Rails` and `::File` on `Rails/FilePath` cop
• Merge pull request #862 from r7kamura/feature/dynamic-find-by-and
• Merge pull request #864 from r7kamura/feature/fix-rubocop-style
• Fix an offense on `Style/RedundantRegexpEscape`
• Ignore if number of arguments does not match on `Rails/DynamicFindBy`
• Use RuboCop RSpec 2.15
• Merge pull request #859 from koic/fix_an_error_for_rails_action_order
• [Fix #841] Fix an error for `Rails/ActionOrder`
• Fix a build error
• Merge pull request #850 from vlad-pisanov/vp_fix_i18n_lazy_lookup
• Merge pull request #851 from koic/fix_false_positive_for_rails_freeze_time
• Merge pull request #857 from r7kamura/feature/action-order-incorrect-correction
• [Fix #837] Fix incorrect autocorrection of `Rails/ActionOrder` about comments
• Merge pull request #855 from gsamokovarov/pathname-parens-autocorrect
• Fix Rails/RootPathnameMethods autocorrection for Pathname calls without parens
• [Fix #848] Fix a false positive for `Rails/FreezeTime`
• Fix default configuration for Rails/I18nLazyLookup
• Bump RuboCop RSpec to 2.14
• Merge pull request #845 from koic/fix_an_incorrect_autocorrect_for_rails_action_order
• [Fix #838] Fix an incorrect autocorrect for `Rails/ActionOrder`
• Merge pull request #844 from koic/fix_a_false_positive_for_rails_action_controller_flash_before_render
• [Fix #843] Fix a false positive for `Rails/ActionControllerFlashBeforeRender`
• 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)

[eduardoj]

@depfu rebase

[eduardoj]

@depfu merge