Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update rubocop requirement from ~> 1.25.0 to >= 1.25, < 1.29 #48

Merged
merged 4 commits into from
Jun 8, 2022

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Apr 21, 2022

Updates the requirements on rubocop to permit the latest version.

Changelog

Sourced from rubocop's changelog.

1.28.0 (2022-04-21)

New features

Bug fixes

  • #10528: Fix an infinite loop at autocorrect for Layout/CaseIndentation. ([@​ydah][])
  • #10537: Fix an incorrect auto-correct for Style/MultilineTernaryOperator when returning a multiline ternary operator expression with break, next, or method call. ([@​koic][])
  • #10529: Fix autocorrect for Style/SoleNestedConditional causes logical error when using a outer condition of method call by omitting parentheses for method arguments. ([@​nobuyo][])
  • #10530: Fix a false positive for Style/RedundantRegexpCharacterClass when using regexp character class with a character class containing multiple unicode code-points. ([@​koic][])
  • #10518: Fix a false positive for Style/DoubleNegation when inside returned conditional clauses with Ruby 2.7's pattern matching. ([@​koic][])
  • #10510: Fix an error for Style/SingleArgumentDig when using multiple dig in a method chain. ([@​koic][])
  • #10553: Fix crash with trailing tabs in heredocs for Layout/TrailingWhitespace. ([@​dvandersluis][])
  • #10488: Fix autocorrection for Layout/MultilineMethodCallIndentation breaks indentation for nesting of method calls. ([@​nobuyo][])
  • #10543: Fix incorrect code length calculation for few more patterns of hash folding asked. ([@​nobuyo][])
  • #10541: Fix an incorrect autocorrect for Style/SpecialGlobalVars when global variable as Perl name is used multiple times. ([@​koic][])
  • #10514: Fix an error for Lint/EmptyConditionalBody when missing second elsif body. ([@​koic][])
  • #10469: Fix code length calculation when kwargs written in single line. ([@​nobuyo][])

Changes

  • #10555: Deprecate IgnoredPatterns in favour of AllowedPatterns. ([@​dvandersluis][])
  • #10356: Add AllowConsecutiveConditionals option to Style/GuardClause and the option is false by default. ([@​ydah][])
  • #10524: Mark Style/RedundantInitialize as unsafe. ([@​koic][])
  • #10280: Add AllowComments option to Style/SymbolProc and the option is false by default. ([@​ydah][])

1.27.0 (2022-04-08)

New features

  • #10500: Add new Lint/RefinementImportMethods cop. ([@​koic][])
  • #10438: Add new Style/RedundantInitialize cop to check for unnecessary initialize methods. ([@​dvandersluis][])

Bug fixes

  • #10464: Fix an incorrect autocorrect for Lint/IncompatibleIoSelectWithFiberScheduler when using IO.select with read (or write) argument and using return value. ([@​koic][])
  • #10506: Fix an error for Style/RaiseArgs when raise with new method without receiver. ([@​koic][])
  • #10479: Fix a false positive for Lint/ShadowingOuterLocalVariable conditional statement and block variable. ([@​ydah][])
  • #10189: Fix --display-style-guide so it works together with --format offenses. ([@​jonas054][])
  • #10465: Fix false positive for Naming/BlockForwarding when the block argument is assigned. ([@​dvandersluis][])
  • #10491: Improve the handling of comments in Lint/EmptyConditionalBody, Lint/EmptyInPattern and Lint/EmptyWhen when AllowComments is set to true. ([@​Darhazer][])

... (truncated)

Commits
  • 24ab7b2 Cut 1.28.0
  • 3e11aa1 Update Changelog
  • 7fcb272 Support autocorrection for Style/RedundantInitialize
  • e491166 Update uses of IgnoredPatterns to AllowedPatterns.
  • 8ac5b59 Add obsoletion rules for AllowedPatterns.
  • 7c9e611 Rename IgnoredPatterns to AllowedPatterns.
  • e62a69a [Fix #10553] Fix crash with trailing tabs in heredocs for `Layout/TrailingWhi...
  • b4ea4c6 [Fix #10280] Add AllowComments option to Style/SymbolProc
  • 7812417 [Fix #10488] Fix autocorrection for Layout/MultilineMethodCallIndentation b...
  • a2cfdc1 Fix an incorrect autocorrect for Style/SpecialGlobalVars
  • Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Updates the requirements on [rubocop](https://github.com/rubocop/rubocop) to permit the latest version.
- [Release notes](https://github.com/rubocop/rubocop/releases)
- [Changelog](https://github.com/rubocop/rubocop/blob/master/CHANGELOG.md)
- [Commits](rubocop/rubocop@v1.25.0...v1.28.0)

---
updated-dependencies:
- dependency-name: rubocop
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Apr 21, 2022
IgnoredPatterns was deprecated in RuboCop v1.28.0 in favor of
AllowedPatterns, which is a little easier to comprehend.
@jnebeker
Copy link
Contributor

For the next dependency meeting (May the 4th!):

  • Resolve the VCR errors in CI
  • Pick up at RuboCop version 1.28.1

@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Apr 28, 2022

A newer version of rubocop exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged.

Mostly replacing ENV[] with ENV.fetch(), also ignore RuboCop for some
`.map {}.to_h` calls that are used in benchmarks.
If we're using consecutive conditionals we probably know what we're
doing. The good thing is this cop will still flag consecutive
conditionals that are identical.
@yarmiganosca yarmiganosca force-pushed the dependabot/bundler/rubocop-gte-1.25-and-lt-1.29 branch from 3f65800 to bf41978 Compare May 4, 2022 16:21
@jcavena
Copy link
Contributor

jcavena commented May 4, 2022

@yarmiganosca You should tell da bot to merge this.

@jcavena jcavena merged commit 27aad0c into main Jun 8, 2022
@jcavena jcavena deleted the dependabot/bundler/rubocop-gte-1.25-and-lt-1.29 branch June 8, 2022 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants