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 and add rubocop-rspec #15
Conversation
[#177398947]
b4d7ef6
to
ebb1a07
Compare
That covers all things Rspec, including lingering `focus`es. [#177398947]
[#177398947]
ebb1a07
to
f8b125f
Compare
spec.add_dependency "rubocop-rspec-focused", "= 0.0.3" | ||
spec.add_dependency "rubocop", "~> 1.18.3" |
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.
I think we might be able to replace https://github.com/charitywater/cw-style/blob/master/lib/unless_with_multiple_conditions.rb with rubocop/rubocop#5388 now.
From skimming https://github.com/rubocop/rubocop/blob/master/CHANGELOG.md
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.
Good catch, thanks! Did that and also ignored a couple more cops that seemed pretty ignorable to me.
We _mostly_ use snakecase for variables with numbers, but not always, and I don't think it's worth the effort to standardize that. [#177398947]
Prettier undoes what autocorrecting this cop does. [#177398947]
We are purposeful about our use of symbols with and without quoted strings and I don't think we need this. [#177398947]
4f30c84
to
58a088b
Compare
Per PR feedback. [#177398947]
58a088b
to
49a74da
Compare
No description provided.