[ruby] Upgrade rubocop to version 1.41.0 #230
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 (~> 1.39.0 → ~> 1.41.0) · Repo · Changelog
Release Notes
1.41.0
Does any of this look wrong? Please let us know.
Commits
See the full diff on Github. The new version differs by 55 commits:
Cut 1.41
Update Changelog
Make `Style/IfWithSemicolon` aware of one line without `else` body
[Fix #11267] Fix an error for Style/RequireOrder when modifier conditional is used between `require`
Mark as private for YARD
Allow extensions inject their defaults
Add new `Style/RedundantDoubleSplatBrackets` cop
Fix `Lint/SafeNavigationChain` to correctly handle `[]` operator followed by save navigation and method chain
More rubocop optimizations
Merge pull request #11301 from fatkodima/memory_profiler-file
Print `memory_profiler` report to the file
Correctly detect Rails version when using only parts of the framework
Fix an error for `Lint/NonAtomicFileOperation`
Fix base_dir in TargetFinder#find_files() (#11299)
[Doc] Fix a typo in the changelog
[Fix #11292] Tweak autocorrection for `Lint/NonAtomicFileOperation`
Merge pull request #11287 from alexmcdermid/patch-1
update class_length.rb and module_length.rb
[Fix #11284] Fix an incorrect autocorrect for `Style/WordArray`
Change `Badge#department` type back to `Symbol`
Add `rubocop-changed` extension to documentation (#11280)
More rubocop optimizations
[Fix #11266] Fix a false positive for `Style/RedundantConstantBase`
[Fix #11277] Add new `Style/ConcatArrayLiterals` cop
Merge pull request #11281 from arika/fix_style_documentation_cop
Fix NoMethodError for Style/Documentation when a class nested under non-constant values
Generate documentation for AllowMultilineFinalElement
Update cops documentation
Introduce AllowMultilineFinalElement option
Use RuboCop RSpec 2.16
[Fix #11273] Fix a false positive for `Lint/DuplicateMethods`
Merge pull request #11276 from ydah/fix/missing_link
Merge pull request #11275 from ydah/add/description_ci
Fixed missing link in CONTRIBUTING.md
Add description of linting YAML files to CONTRIBUTING.md
More rubocop optimizations
Fix PR identifier for `Style/RequireOrder` cop
Merge pull request #11257 from koic/fix_an_incorrect_autocorrect_for_style_hash_syntax
Tweak an error message when configuring `Lint/Syntax`
Merge pull request #11262 from fatkodima/if_unless_modifier-splat
Merge pull request #11263 from ydah/fix/typos
Fix a typo
Fix an error for `Style/IfUnlessModifier` when the body is a method call with hash splat
Merge pull request #11258 from ydah/fix/error_require_order
[Fix #11255] Fix an error for `Style/RequireOrder` when `require` with no arguments is put between `require`
Merge pull request #11254 from koic/fix_an_error_for_style_require_order
[Fix #11256] Fix an incorrect autocorrect for `Style/HashSyntax`
Fix an error for `Style/RequireOrder`
Tweak optimized checking if the file is relevant for the cop
Remove a redundant guard clause
Merge pull request #11250 from ydah/fix/error_guard_clause
Fix an error for `Style/GuardClause` when a method call whose last argument is not a string is in the condition body
Fix a typo
Fix a build error
Switch back the docs version
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