-
Notifications
You must be signed in to change notification settings - Fork 6
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 from 0.47.1 to 0.60.0 #28
Conversation
dleve123
commented
Nov 6, 2018
- Addresses CVE-2017-8418
- Re-generates .rubocop_todo.yml to skip failing style validations
Danger run resulted in 2 warnings; to find out more, see the checks page. Generated by 🚫 dangerJS |
Here's an example of a CHANGELOG.md entry: * [#28](https://github.com/artsy/estella/pull/28): Update rubocop from 0.47.1 to 0.60.0 - [@dleve123](https://github.com/dleve123). Generated by 🚫 Danger |
Interesting build failure - looks like it's related to Elasticsearch API updates. We probably need to freeze the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you run “rubocop -a”? I would do that before generating new configuration.
* run a `rubocop -a` and commit all automatically generated style changes
3885615
to
9ac5f8c
Compare
* elasticsearch-models's 2.X series supports Elasticsearch 2.X. + See [elasticsearch-model's README][0] for more info [0]:https://github.com/elastic/elasticsearch-rails/tree/4429fd9c27b4808a721e90481fd8814a0b7486d4/elasticsearch-model#compatibility
@dblock @cavvia – this PR should be good to go. There's no meaningful diff to the project, so I'm not sure what Coveralls is complaining about. LMK if there's anything else needed to get this merged. FWIW: rubocop is a development dependency, so there is no additional upstream systems that need to get patched here. |
Merged. |