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

Support for rubocop 0.92 via rubocop-shopify #190

Closed
wants to merge 1 commit into from

Conversation

aried3r
Copy link

@aried3r aried3r commented Oct 22, 2020

v1.0.6 has explicit support for 0.92 which should be a good step towards
rubocop 1.0.0 support which was released recently.

https://github.com/Shopify/ruby-style-guide/releases/tag/v1.0.6

Support for rubocop 1.0.0 should land soon as well:
Shopify/ruby-style-guide#207

v1.0.6 has explicit support for 0.92 which should be a good step towards
rubocop 1.0.0 support which was released recently.

https://github.com/Shopify/ruby-style-guide/releases/tag/v1.0.6

Support for rubocop 1.0.0 should land soon as well:
Shopify/ruby-style-guide#207
@aried3r
Copy link
Author

aried3r commented Oct 30, 2020

I can reproduce the CI errors locally, but I'm not sure how to best resolve them. Using rubocop-shopify 1.0.4 which has support for rubocop 0.86 it seems to just not work properly. Is it a stated goal to keep supporting old rubocops?

@afdev82
Copy link

afdev82 commented Nov 4, 2020

Any news about that? In our project we would like to update to rubocop v1, but we can't because of erb_lint still requires old rubocop versions.
Thank you very much for your support!

@rafaelfranca
Copy link
Member

Trying to understand this. The version of rubocop is not locked in erb-lint (see https://github.com/Shopify/erb-lint/blob/540b88a9a79720930a62bdd2a05419a6b59d9e3d/erb_lint.gemspec#L29) so it should not be preventing you to upgrade. Do you mean that it is not working with the newest version or it is not allowing to install the latest version?

@rafaelfranca
Copy link
Member

Our CI is testing with Rubocop 1.10 already and it is passing. So I'm closing this PR. Thank you so much.

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.

4 participants