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 version to 0.67 #1867

Merged
merged 3 commits into from
Apr 16, 2019
Merged

Bump RuboCop version to 0.67 #1867

merged 3 commits into from
Apr 16, 2019

Conversation

yahonda
Copy link
Collaborator

@yahonda yahonda commented Apr 16, 2019

* This warning appears without this change

```ruby
[Warn] Performance Cops will be removed from RuboCop 0.68. Use rubocop-performance gem instead.
       https://github.com/rubocop-hq/rubocop/tree/master/manual/migrate_performance_cops.md

Inspecting 66 files
..................................................................

66 files inspected, no offenses detected
```

* There are seven Performance Cops enabled

```
$ grep Performance .rubocop.yml
Performance/FlatMap:
Performance/RedundantMerge:
Performance/StartWith:
Performance/EndWith:
Performance/RegexpMatch:
Performance/ReverseEach:
Performance/UnfreezeString:
$
```
@yahonda yahonda requested a review from koic April 16, 2019 00:28
@yahonda
Copy link
Collaborator Author

yahonda commented Apr 16, 2019

@koic Would you review this pull request?

@koic
Copy link
Collaborator

koic commented Apr 16, 2019

Thank you for catching up. The changes looks good to me.

I think that it follows the version of RuboCop used by Rails. I opened a PR rails/rails#35989 to Rails repo. I'd like to merge this PR when rails/rails#35989 will be merged.

Copy link
Collaborator

@koic koic left a comment

Choose a reason for hiding this comment

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

rails/rails#35989 was merged. Thank you!

@koic koic merged commit b73a257 into rsim:master Apr 16, 2019
@yahonda
Copy link
Collaborator Author

yahonda commented Apr 16, 2019

Thanks for the review and pull request for Rails.

@yahonda yahonda deleted the rubocop_067 branch April 18, 2019 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants