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
Fix Rubocop 0.81.0 compatibility #146
Conversation
I suppose we'll need to release a v1.0.1 of this gem once this is merged. 🤔 |
Yes we will. Also given that RuboCop ships breaking changes in minor versions we should limit the range of the compatible version of RuboCop to allow automatic patch upgrades only (e.g. |
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.
We need to change the rubocop version in the gemspec since the minimum version is 0.81.0
fb33635
to
1770c92
Compare
Great points. I've updated the |
1770c92
to
1b48316
Compare
Rubocop v0.81.0 removed
Lint/EndInMethod
in favour of consolidating withinStyle/EngBlock
. (rubocop/rubocop#7813)Thus, I removed our
Lint/EndInMethod
rule. We already haveStyle/EngBlock
specified here:ruby-style-guide/rubocop.yml
Lines 635 to 636 in b5e4bc9
Example failure using
rubocop-shopify-1.0.0
: