Skip to content

Update rubocop: 1.11.0 → 1.12.1 (minor)

Depfu Bot requested to merge depfu/update/rubocop-1.12.1 into master

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 (1.11.0 → 1.12.1) · Repo · Changelog
Release Notes

1.12.1

Bug fixes

  • #9649: Fix when highlights contain multibyte characters. (@osyo-manga)
  • #9646: Fix an incorrect auto-correct for EnforcedStyle: require_parentheses of Style/MethodCallWithArgsParentheses with EnforcedStyle: conditionals of Style/AndOr. (@koic)
  • #9608: Fix a false positive for Layout/EmptyLineAfterGuardClause when using guard clause is after rubocop:enable comment. (@koic)
  • #9637: Allow parentheses for forwarded args in Style/MethodCallWithArgsParentheses's omit_parentheses style to avoid endless range ambiguity. (@gsamokovarov)
  • #9641: Fix Layout/MultilineMethodCallIndentation triggering on method calls that look like operators. (@dvandersluis)
  • #9638: Fix an error for Layout/LineLength when over limit at right hand side of multiple assignment. (@koic)
  • #9639: Fix Style/RedundantBegin removing comments on assignment statement correction. (@marcotc)
  • #9671: Fix an incorrect auto-correct for Lint/AmbiguousOperator with Style/MethodCallWithArgsParentheses. (@koic)
  • #9645: Fix an incorrect auto-correct for Style/SingleLineMethods when using single line class method definition. (@koic)
  • #9644: Fix an error and an incorrect auto-correct for Style/MultilineMethodSignature when line break after opening parenthesis. (@koic)
  • #9672: Fix an incorrect auto-correct for Style/HashConversion when using multi-argument Hash[] as a method argument. (@koic)

1.12.0

New features

  • #9615: Add new Style/StringChars cop. (@koic)
  • #9629: Add AllowParenthesesInStringInterpolation configuration to Style/MethodCallWithArgsParentheses to allow parenthesized calls in string interpolation. (@gsamokovarov)
  • #9219: Allow excluding some constants from Style/Documentation. (@fsateler)
  • Add AllowNil option for Lint/SuppressedException to allow/disallow rescue nil. (@corroded)

Bug fixes

  • #9560: Fix an error for Lint/ClassMethodsDefinitions when defining class methods with class << self with comment only body. (@koic)
  • #9551: Fix a false positive for Style/UnlessLogicalOperators when using || operator and invoked method name includes "or" in the conditional branch. (@koic)
  • #9620: Allow parentheses in operator methods calls for Style/MethodCallWithArgsParentheses EnforcedStyle: omit_parentheses. (@gsamokovarov)
  • #9622: Fixed Style/BisectedAttrAccessor autocorrection to handle multiple bisected attrs in the same macro. (@dvandersluis)
  • #9606: Fix an error for Layout/IndentationConsistency when using access modifier at the top level. (@koic)
  • #9619: Fix infinite loop between Layout/IndentationWidth and Layout/RescueEnsureAlignment autocorrection. (@dvandersluis)
  • #9633: Fix an incorrect auto-correct for Lint/NumberConversion when to_i method in symbol form. (@koic)
  • #9616: Fix an incorrect auto-correct for Style/EvalWithLocation when using #instance_eval with a string argument in parentheses. (@koic)
  • #9429: Fix Style/NegatedIfElseCondition autocorrect to keep comments in correct branch. (@tejasbubane)
  • #9631: Fix an incorrect auto-correct for Style/RedundantReturn when using return with splat argument. (@koic)
  • #9627: Fix an incorrect auto-correct for Style/StructInheritance when extending instance of Struct without do ... end and class body is empty. (@koic)
  • #5953: Fix a false positive for Style/AccessModifierDeclarations when using module_function with symbol. (@koic)
  • #9593: Fix an error when processing a directory is named {}. (@koic)
  • #9599: Fix an error for Style/CaseLikeIf when using include? without a receiver. (@koic)
  • #9582: Fix incorrect auto-correct for Style/ClassEqualityComparison when comparing Module#name for equality. (@koic)
  • #9603: Fix a false positive for Style/SoleNestedConditional when using nested modifier on value assigned in condition. (@koic)
  • #9598: Fix RuboCop::MagicComment#valid_shareable_constant_value?. (@kachick)
  • #9625: Allow parentheses in yield arguments with Style/MethodCallWithArgsParentheses EnforcedStyle: omit_parentheses to fix invalid Ruby auto-correction. (@gsamokovarov)
  • #9558: Fix inconsistency when dealing with URIs that are wrapped in single quotes vs double quotes. (@dvandersluis)
  • #9613: Fix a false positive for Style/RedundantSelf when a self receiver on an lvalue of mlhs arguments. (@koic)
  • #9586: Update Naming/RescuedExceptionsVariableName to not register on inner rescues when nested. (@dvandersluis)

Changes

  • #9487: Mark Naming/MemoizedInstanceVariableName as unsafe. (@marcandre)
  • #9601: Make Style/RedundantBegin aware of redundant begin/end blocks around memoization. (@koic)
  • #9617: Disable suggested extensions when using the --stdin option. (@dvandersluis)

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

Commits

See the full diff on Github. The new version differs by more commits than we can show here.

↗️ rexml (indirect, 3.2.4 → 3.2.5) · Repo · Changelog
Commits

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

Depfu Status

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)

Merge request reports