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 to 0.68.1 #36176

Conversation

natesholland
Copy link

Summary

Performance cops were extracted in rubocop/rubocop#5977 and @koic
updated to 0.67.2 and added the rubocop-performance gem in #35989.

This bumps rubocop to 0.68.1 to be on the latest version. Two changes
were made in regards to the bump of the version:

Performance cops were extracted in rubocop/rubocop#5977 and @koic
updated to 0.67.2 and added the rubocop-performance gem in rails#35989.

This bumps rubocop to 0.68.1 to be on the latest version. Two changes
were made in regards to the bump of the version:
* The first was that Layout/FirstParameterIndentation was renamed to
Layout/IndentFirstArgument. These changes can be seen in
rubocop/rubocop#6982 and rubocop/rubocop#6987
* The second was that a new instance of the Style/HashSyntax cop was
found. I updated that instance to follow the new Ruby >= 1.9 syntax.
@rails-bot rails-bot bot added the actionview label May 4, 2019
@y-yagi
Copy link
Member

y-yagi commented May 4, 2019

Thanks for your PR.
We need to use the same version in CodeClimate. But CodeClimate does not support 0.68 yet.
https://github.com/codeclimate/codeclimate/blob/e31c097b4069b84106c4c074be62e05fce96d801/config/engines.yml#L177-L201
Ref: #36151.

Please try to update the CodeClimate side first. Thanks.

@y-yagi y-yagi closed this May 4, 2019
@natesholland
Copy link
Author

Working on it here: codeclimate/codeclimate-rubocop#168

@koic
Copy link
Contributor

koic commented May 4, 2019

FYI, please bump RuboCop Performance to 1.2.0 to apply Performance/RegexpMatch false negative fix when upgrading to RuboCop 0.68.
rubocop/rubocop-performance#47

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants