Update rubocop: 0.66.0 → 0.67.2 (major) #526
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 update. Please take a good look at what changed and the test results before merging this pull request.
What changed?
✳️ rubocop (0.66.0 → 0.67.2) · Repo · Changelog
Release Notes
0.67.2
0.67.1
0.67.0
Does any of this look wrong? Please let us know.
Commits
See the full diff on Github. The new version differs by 41 commits:
Cut 0.67.2
Fix a typo
[Fix #6882] Fix an error for `Rails/RedundantAllowNil`
Cut 0.67.1
Set default `PreferredName` to `e` for `Naming/RescuedExceptionsVariableName`
Cut 0.67
Fix the manual
Add new Naming/RescuedExceptionsVariableName cop
[Fix #6785] Make Rails/Blank and Rails/Present aware of Style/UnlessElse
Update a bit of cop metadata
[Fix #5184] Add new multiline element line break cops (#6824)
Add `STDERR.puts` check to Style/StderrPuts cop
[Fix #6871] Move several performance cops to the style department
Insert a newline at EOF of CHANGELOG.md
Mention block form of Struct.new in Style/StructInheritance (#6875)
Add IncludedMacros param to default rubocop config for MethodCallWithArgsParentheses
Mark Rails/LexicallyScopedActionFilter as unsafe
Clarify dangers of Rails/LexicallyScopedActionFilter
Add new cop `Rails/ActiveRecordOverride`
NodePattern: Allow numerals in identifiers
[Fix #6761] Make UncommunicativeMethodParamName account for underscores (#6872)
Add `Rails/RedundantAllowNil` cop
Fix false positive for `Rails/LinkToBlank` when rel is a symbol value
Merge pull request #6874 from koic/rename_files_for_style_strip
Rename files for `Style/Strip` cop
Rename Style/LstripRstrip to Style/Strip
[Fix #6637] promoted Performance/LstripRstrip to Style/LstripRstrip
Fix Rails/LinkToBlank auto-correct bug when using symbol for target
[User Manual] Alphabetize CLI options (#6867)
Merge pull request #6865 from aeroastro/feature/remove-preceding-prefix
Fix deactivated `StyleGuideBaseURL` for `Layout/ClassStructure`
Merge pull request #6859 from koic/fix_incorrect_autocorrect_for_lint_to_json
[Fix #6858] Fix an incorrect auto-correct for `Lint/ToJSON`
Merge pull request #6856 from annkissam/handle_exception_in_block_comments
handle autocorrect exception in Style/BlockComments
handle exception in Rails/RedundantReceiverInWithOptions with empty body (#6855)
Merge pull request #6852 from iliabylich/use-absolute-path-to-gemfile-lock
Merge pull request #6851 from marcandre/type_pattern
Use absolute path to Gemfile.local in Gemfile.
Fix NodePattern of node type on a literal
[#5977] Warn for Performance Cops
Commits
See the full diff on Github. The new version differs by 36 commits:
v1.17.0
Merge pull request #251 from Yuki-Inoue/control-timing-of-async-exception
Merge pull request #240 from walf443/update_mysql2
Merge branch 'master' into update_mysql2
Merge pull request #252 from grosser/grosser/test
try to run isolation again
v1.16.2
Merge pull request #253 from grosser/grosser/lap
prefer local variable over instance variable
Merge pull request #255 from grosser/grosser/lock
lock file gets checked in a lot, just ignore it on travis
Merge pull request #254 from aeroastro/feature/travis-badge-for-master
Migrate travis badge from PNG to SVG
v1.16.1
thx for the fix
Merge pull request #245 from aeroastro/feature/fix-nil-worker
Calculate lap correctly to replace worker in isolation mode
fix signature
control timing of async exception
Merge pull request #250 from aeroastro/feature/fix-travis-build-by-removing-bundle-lock
Fix travis failure by removing bundled version
thx for the PR
v1.16.0
Merge pull request #248 from Yuki-Inoue/ensure-kill-child-threads
process
threads kill instant test
Merge pull request #249 from Yuki-Inoue/freeze-stop
freeze Stop object
child gracefully halt when parent already died
not map! but map
ensure to kill threads
Merge pull request #247 from grosser/grosser/fix
fix travis
v1.15.0
Merge pull request #244 from grosser/grosser/isolation
avoid nil errors by initializing all workers during isolation runs
Release Notes
2.6.2.0 (from changelog)
Does any of this look wrong? Please let us know.
Commits
See the full diff on Github. The new version differs by 17 commits:
Update changelog.
Bump version.
* Bump 2.4 branch to 2.4.6. (#569)
Add missing node types to Meta::NODE_TYPES
* Lexer should know about current parsing context. (#566)
Update changelog.
Update changelog.
Bump version.
* Bump ruby versions to 2.5.5 and 2.6.2. (#563)
+ ruby27.y: dsym should be treated as string. (#560)
+ ruby27.y: Refactored symbol rules. (#557)
+ ruby27.y: Added method reference operator. (#556)
* Bump Ruby version to 2.6.1. (#554)
Remove unreachable code from lexer.rl (#552)
+ ruby27.y: branch parser. (#546)
Fix example in TreeRewriter documentation (#543)
Update changelog.
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