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

Bump rubocop from 0.60.0 to 0.75.0 #3739

Merged
merged 4 commits into from
Oct 8, 2019
Merged

Conversation

dependabot-preview[bot]
Copy link
Contributor

Bumps rubocop from 0.60.0 to 0.75.0.

Release notes

Sourced from rubocop's releases.

RuboCop 0.75

New features

  • #7274: Add new Lint/SendWithMixinArgument cop. (@​koic)
  • #7272: Show warning message if passed string to Enabled, Safe, SafeAutocorrect, and AutoCorrect keys in .rubocop.yml. ([@​unasuke][])
  • #7295: Make it possible to set StyleGuideBaseURL per department. (@​koic)
  • #7301: Add check for calls to remote_byebug to Lint/Debugger cop. ([@​riley-klingler][])
  • #7321: Allow YAML aliases in .rubocop.yml. ([@​raymondfallon][])
  • #7317: Add new formatter pacman. ([@​crojasaragonez][])
  • #6075: Support IgnoredPatterns option for Naming/MethodName cop. (@​koic)
  • #7335: Add todo as an alias to disable. --disable-uncorrectable will now disable cops using rubocop:todo instead of rubocop:disable. ([@​desheikh][])

Bug fixes

  • #7256: Fix an error of Style/RedundantParentheses on method calls where the first argument begins with a hash literal. ([@​halfwhole][])
  • #7263: Make Layout/SpaceInsideArrayLiteralBrackets properly handle tab-indented arrays. ([@​buehmann][])
  • #7252: Prevent infinite loops by making Layout/SpaceInsideStringInterpolation skip over interpolations that start or end with a line break. ([@​buehmann][])
  • #7262: Lint/FormatParameterMismatch did not recognize named format sequences like %.2<name>f where the name appears after some modifiers. ([@​buehmann][])
  • #7253: Fix an error for Lint/NumberConversion when #to_i called without a receiver. (@​koic)
  • #7271, #6498: Fix an interference between Style/TrailingCommaIn*Literal and Layout/Multiline*BraceLayout for arrays and hashes. ([@​buehmann][])
  • #7241: Make Style/FrozenStringLiteralComment match only true & false. ([@​tejasbubane][])
  • #7290: Handle inner conditional inside else in Style/ConditionalAssignment. ([@​jonas054][])
  • #5788: Allow block arguments on separate lines if line would be too long in Layout/MultilineBlockLayout. ([@​jonas054][])
  • #7305: Register Style/BlockDelimiters offense when block result is assigned to an attribute. ([@​mvz][])
  • #4802: Don't leave any Lint/UnneededCopEnableDirective offenses undetected/uncorrected. ([@​jonas054][])
  • #7326: Fix a false positive for Style/AccessModifierDeclarations when access modifier name is used for hash literal value. (@​koic)
  • #3591: Handle modifier if/unless correctly in Lint/UselessAssignment. ([@​jonas054][])
  • #7161: Fix Style/SafeNavigation cop for preserve comments inside if expression. ([@​tejasbubane][])
  • #5212: Avoid false positive for braces that are needed to preserve semantics in Style/BracesAroundHashParameters. ([@​jonas054][])
  • #7353: Fix a false positive for Style/RedundantSelf when receiver and multiple assigned lvalue have the same name. (@​koic)
  • #7353: Fix a false positive for Style/RedundantSelf when a self receiver is used as a method argument. (@​koic)
  • #7358: Fix an incorrect autocorrect for Style/NestedModifier when parentheses are required in method arguments. (@​koic)
  • #7361: Fix a false positive for Style/TernaryParentheses when only the closing parenthesis is used in the last line of condition. (@​koic)
  • #7369: Fix an infinite loop error for Layout/IndentAssignment with Layout/IndentFirstArgument when using multiple assignment. (@​koic)
  • #7177, #7370: When correcting alignment, do not insert spaces into string literals. ([@​buehmann][])
  • #7367: Fix an error for Style/OrAssignment cop when then branch body is empty. (@​koic)
  • #7363: Fix an incorrect autocorrect for Layout/SpaceInsideBlockBraces and Style/BlockDelimiters when using multiline empty braces. (@​koic)
  • #7212: Fix a false positive for Layout/EmptyLinesAroundAccessModifier and UselessAccessModifier when using method with the same name as access modifier around a method definition. (@​koic)

Changes

  • #7312: Mark Style/StringHashKeys as unsafe. ([@​prathamesh-sonpatki][])
  • #7275: Make Style/VariableName aware argument names when invoking a method. (@​koic)
  • #3534: Make Style/IfUnlessModifier report and auto-correct modifier lines that are too long. ([@​jonas054][])
  • #7261: Style/FrozenStringLiteralComment no longer inserts an empty line after the comment. This is left to Layout/EmptyLineAfterMagicComment. ([@​buehmann][])
  • #7091: Style/FormatStringToken now detects format sequences with flags and modifiers. ([@​buehmann][])
  • #7319: Rename IgnoredMethodPatterns option to IgnoredPatterns option for Style/MethodCallWithArgsParentheses. (@​koic)
  • #7345: Mark unsafe for Style/YodaCondition. (@​koic)
... (truncated)
Changelog

Sourced from rubocop's changelog.

0.75.0 (2019-09-30)

New features

  • #7274: Add new Lint/SendWithMixinArgument cop. (@​koic)
  • #7272: Show warning message if passed string to Enabled, Safe, SafeAutocorrect, and AutoCorrect keys in .rubocop.yml. ([@​unasuke][])
  • #7295: Make it possible to set StyleGuideBaseURL per department. (@​koic)
  • #7301: Add check for calls to remote_byebug to Lint/Debugger cop. ([@​riley-klingler][])
  • #7321: Allow YAML aliases in .rubocop.yml. ([@​raymondfallon][])
  • #7317: Add new formatter pacman. ([@​crojasaragonez][])
  • #6075: Support IgnoredPatterns option for Naming/MethodName cop. (@​koic)
  • #7335: Add todo as an alias to disable. --disable-uncorrectable will now disable cops using rubocop:todo instead of rubocop:disable. ([@​desheikh][])

Bug fixes

  • #7256: Fix an error of Style/RedundantParentheses on method calls where the first argument begins with a hash literal. ([@​halfwhole][])
  • #7263: Make Layout/SpaceInsideArrayLiteralBrackets properly handle tab-indented arrays. ([@​buehmann][])
  • #7252: Prevent infinite loops by making Layout/SpaceInsideStringInterpolation skip over interpolations that start or end with a line break. ([@​buehmann][])
  • #7262: Lint/FormatParameterMismatch did not recognize named format sequences like %.2<name>f where the name appears after some modifiers. ([@​buehmann][])
  • #7253: Fix an error for Lint/NumberConversion when #to_i called without a receiver. (@​koic)
  • #7271, #6498: Fix an interference between Style/TrailingCommaIn*Literal and Layout/Multiline*BraceLayout for arrays and hashes. ([@​buehmann][])
  • #7241: Make Style/FrozenStringLiteralComment match only true & false. ([@​tejasbubane][])
  • #7290: Handle inner conditional inside else in Style/ConditionalAssignment. ([@​jonas054][])
  • #5788: Allow block arguments on separate lines if line would be too long in Layout/MultilineBlockLayout. ([@​jonas054][])
  • #7305: Register Style/BlockDelimiters offense when block result is assigned to an attribute. ([@​mvz][])
  • #4802: Don't leave any Lint/UnneededCopEnableDirective offenses undetected/uncorrected. ([@​jonas054][])
  • #7326: Fix a false positive for Style/AccessModifierDeclarations when access modifier name is used for hash literal value. (@​koic)
  • #3591: Handle modifier if/unless correctly in Lint/UselessAssignment. ([@​jonas054][])
  • #7161: Fix Style/SafeNavigation cop for preserve comments inside if expression. ([@​tejasbubane][])
  • #5212: Avoid false positive for braces that are needed to preserve semantics in Style/BracesAroundHashParameters. ([@​jonas054][])
  • #7353: Fix a false positive for Style/RedundantSelf when receiver and multiple assigned lvalue have the same name. (@​koic)
  • #7353: Fix a false positive for Style/RedundantSelf when a self receiver is used as a method argument. (@​koic)
  • #7358: Fix an incorrect autocorrect for Style/NestedModifier when parentheses are required in method arguments. (@​koic)
  • #7361: Fix a false positive for Style/TernaryParentheses when only the closing parenthesis is used in the last line of condition. (@​koic)
  • #7369: Fix an infinite loop error for Layout/IndentAssignment with Layout/IndentFirstArgument when using multiple assignment. (@​koic)
  • #7177, #7370: When correcting alignment, do not insert spaces into string literals. ([@​buehmann][])
  • #7367: Fix an error for Style/OrAssignment cop when then branch body is empty. (@​koic)
  • #7363: Fix an incorrect autocorrect for Layout/SpaceInsideBlockBraces and Style/BlockDelimiters when using multiline empty braces. (@​koic)
  • #7212: Fix a false positive for Layout/EmptyLinesAroundAccessModifier and UselessAccessModifier when using method with the same name as access modifier around a method definition. (@​koic)

Changes

  • #7312: Mark Style/StringHashKeys as unsafe. ([@​prathamesh-sonpatki][])
  • #7275: Make Style/VariableName aware argument names when invoking a method. (@​koic)
  • #3534: Make Style/IfUnlessModifier report and auto-correct modifier lines that are too long. ([@​jonas054][])
  • #7261: Style/FrozenStringLiteralComment no longer inserts an empty line after the comment. This is left to Layout/EmptyLineAfterMagicComment. ([@​buehmann][])
  • #7091: Style/FormatStringToken now detects format sequences with flags and modifiers. ([@​buehmann][])
  • #7319: Rename IgnoredMethodPatterns option to IgnoredPatterns option for Style/MethodCallWithArgsParentheses. (@​koic)
  • #7345: Mark unsafe for Style/YodaCondition. (@​koic)
... (truncated)
Commits

Dependabot compatibility score

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)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language
  • @dependabot badge me will comment on this PR with code to add a "Dependabot enabled" badge to your readme

Additionally, you can set the following in your Dependabot dashboard:

  • Update frequency (including time of day and day of week)
  • Pull request limits (per update run and/or open at any time)
  • Automerge options (never/patch/minor, and dev/runtime dependencies)
  • Out-of-range updates (receive only lockfile updates, if desired)
  • Security updates (receive only security updates, if desired)

Finally, you can contact us by mentioning @dependabot.

Bumps [rubocop](https://github.com/rubocop-hq/rubocop) from 0.60.0 to 0.75.0.
- [Release notes](https://github.com/rubocop-hq/rubocop/releases)
- [Changelog](https://github.com/rubocop-hq/rubocop/blob/master/CHANGELOG.md)
- [Commits](rubocop/rubocop@v0.60.0...v0.75.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
@dependabot-preview dependabot-preview bot added the dependencies Pull requests that updates a dependency label Oct 1, 2019
The indentation mode has been renamed.
Copy link
Member

@javierm javierm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note these changes would break our current rubocop configuration. We'll have to install rubocop-rails and update some rules.

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some files could not be reviewed due to errors:

cannot load such file -- rubocop-performance
cannot load such file -- rubocop-performance
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader_resolver.rb:15:in `require'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader_resolver.rb:15:in `block in resolve_requires'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader_resolver.rb:11:in `each'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader_resolver.rb:11:in `resolve_requires'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader.rb:43:in `load_file'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader.rb:83:in `configuration_from_file'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_store.rb:44:in `for'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/cli.rb:181:in `apply_default_formatter'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/cli.rb:40:in `run'
/home/linters/.bundle/gems/rubocop-0.54.0/bin/rubocop:13:in `block in '
/usr/local/lib/ruby/2.6.0/benchmark.rb:308:in `realtime'
/home/linters/.bundle/gems/rubocop-0.54.0/bin/rubocop:12:in `'
/home/linters/.bundle/bin/rubocop:23:in `load'
/home/linters/.bundle/bin/rubocop:23:in `'
/usr/local/lib/ruby/2.6.0/bundler/cli/exec.rb:74:in `load'
/usr/local/lib/ruby/2.6.0/bundler/cli/exec.rb:74:in `kernel_load'
/usr/local/lib/ruby/2.6.0/bundler/cli/exec.rb:28:in `run'
/usr/local/lib/ruby/2.6.0/bundler/cli.rb:463:in `exec'
/usr/local/lib/ruby/2.6.0/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
/usr/local/lib/ruby/2.6.0/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
/usr/local/lib/ruby/2.6.0/bundler/vendor/thor/lib/thor.rb:387:in `dispatch'
/usr/local/lib/ruby/2.6.0/bundler/cli.rb:27:in `dispatch'
/usr/local/lib/ruby/2.6.0/bundler/vendor/thor/lib/thor/base.rb:466:in `start'
/usr/local/lib/ruby/2.6.0/bundler/cli.rb:18:in `start'
/usr/local/lib/ruby/gems/2.6.0/gems/bundler-1.17.2/exe/bundle:30:in `block in '
/usr/local/lib/ruby/2.6.0/bundler/friendly_errors.rb:124:in `with_friendly_errors'
/usr/local/lib/ruby/gems/2.6.0/gems/bundler-1.17.2/exe/bundle:22:in `'
/usr/local/bin/bundle:23:in `load'
/usr/local/bin/bundle:23:in `'

@javierm
Copy link
Member

javierm commented Oct 3, 2019

It looks like after the latest changes, all Rails cops are enabled, while with version 0.60.0 only the ones we specified were enabled.

@javierm
Copy link
Member

javierm commented Oct 8, 2019

Actually, it looks like our current configuration is the one ignoring Rails Cops 😅, and updating will fix it.

In the previous configuration, we forgot to add these lines to rubocop's configuration file:

Rails:
  Enabled: true

Rails and performance cops have been extracted to separate gems.

Note in the past we had to add these lines in order to activate Rails
cops:

```
Rails:
  Enabled: true
```

But we didn't do it, and so Rails cops were ignored.

With the new version, it's enough to require `rubocop-rails`.
@javierm javierm force-pushed the dependabot/bundler/rubocop-0.75.0 branch from 829b75e to 63d31f4 Compare October 8, 2019 10:42
Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some files could not be reviewed due to errors:

cannot load such file -- rubocop-performance
cannot load such file -- rubocop-performance
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader_resolver.rb:15:in `require'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader_resolver.rb:15:in `block in resolve_requires'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader_resolver.rb:11:in `each'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader_resolver.rb:11:in `resolve_requires'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader.rb:43:in `load_file'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_loader.rb:83:in `configuration_from_file'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/config_store.rb:44:in `for'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/cli.rb:181:in `apply_default_formatter'
/home/linters/.bundle/gems/rubocop-0.54.0/lib/rubocop/cli.rb:40:in `run'
/home/linters/.bundle/gems/rubocop-0.54.0/bin/rubocop:13:in `block in '
/usr/local/lib/ruby/2.6.0/benchmark.rb:308:in `realtime'
/home/linters/.bundle/gems/rubocop-0.54.0/bin/rubocop:12:in `'
/home/linters/.bundle/bin/rubocop:23:in `load'
/home/linters/.bundle/bin/rubocop:23:in `'
/usr/local/lib/ruby/2.6.0/bundler/cli/exec.rb:74:in `load'
/usr/local/lib/ruby/2.6.0/bundler/cli/exec.rb:74:in `kernel_load'
/usr/local/lib/ruby/2.6.0/bundler/cli/exec.rb:28:in `run'
/usr/local/lib/ruby/2.6.0/bundler/cli.rb:463:in `exec'
/usr/local/lib/ruby/2.6.0/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
/usr/local/lib/ruby/2.6.0/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
/usr/local/lib/ruby/2.6.0/bundler/vendor/thor/lib/thor.rb:387:in `dispatch'
/usr/local/lib/ruby/2.6.0/bundler/cli.rb:27:in `dispatch'
/usr/local/lib/ruby/2.6.0/bundler/vendor/thor/lib/thor/base.rb:466:in `start'
/usr/local/lib/ruby/2.6.0/bundler/cli.rb:18:in `start'
/usr/local/lib/ruby/gems/2.6.0/gems/bundler-1.17.2/exe/bundle:30:in `block in '
/usr/local/lib/ruby/2.6.0/bundler/friendly_errors.rb:124:in `with_friendly_errors'
/usr/local/lib/ruby/gems/2.6.0/gems/bundler-1.17.2/exe/bundle:22:in `'
/usr/local/bin/bundle:23:in `load'
/usr/local/bin/bundle:23:in `'

Copy link

@houndci-bot houndci-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some files could not be reviewed due to errors:

The configured rubocop version is not supported.
The configured rubocop version is not supported. 
See a list of supported versions on our docs page.

@javierm javierm added this to Reviewing in Roadmap via automation Oct 8, 2019
@javierm javierm moved this from Reviewing to Testing in Roadmap Oct 8, 2019
Hound doesn't support version 0.75 yet, so we need to use 0.72.0.
@javierm javierm force-pushed the dependabot/bundler/rubocop-0.75.0 branch from 2dc0999 to 1955544 Compare October 8, 2019 10:52
@javierm javierm merged commit 2c80c05 into master Oct 8, 2019
Roadmap automation moved this from Testing to Release 1.1.0 Oct 8, 2019
@javierm javierm deleted the dependabot/bundler/rubocop-0.75.0 branch October 8, 2019 11:20
@javierm javierm self-assigned this Oct 9, 2019
smarques pushed a commit to venetochevogliamo/consul that referenced this pull request Apr 29, 2020
…r/rubocop-0.75.0

Bump rubocop from 0.60.0 to 0.75.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that updates a dependency
Projects
No open projects
Roadmap
  
Release 1.1.0
Development

Successfully merging this pull request may close these issues.

None yet

2 participants