-
Notifications
You must be signed in to change notification settings - Fork 1
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
Stronger rubocop version pin prevents upgrading this gem for us #56
Comments
The less strict version pin was actually a non-desired behavior that was introduced by mistake. Do you have a reason to keep two apparently parallel used rubocops in your projects, or why do you have the necessity to upgrade rubocop independently from this gem? |
We recently released version 15.0.0 which increased the rubocop version to 1.70.0. Upgrading the makandra-rubocop should fix your issues. |
This issue originated from the Introducing support for Ruby 3.4 introduced the warnings for the old As this is an edge case, I can live with the current situation and you can consider this issue closed. However, I still don't understand the justification for the strict version pin. |
The purpose of this gem is to have a curated list of cops to save its users the time it takes to make decisions on newly introduced cops themselves and to keep styles consistent across the projects that use it. Pinning rubocop to the less strict |
Thanks for the explanation. |
rubocop
1.61
has problems withstringio
for us, so we upgraded to rubocop1.70
.makandra-rubocop
1.14.0
prints warnings to the console, so we wanted to upgrade to1.14.1
which doesn't print the warnings.However, it has a stricter version pin on rubocop in the gemspec.
~> 1.61.0
instead of~> 1.61
. This is incompatible with newer rubocops.Why was this changed? Was there any known incompatibility? If possible, please use a less strict pin that doesn't prevent rubocop upgrades.
Our temporary fix is to use the old version of makandra-rubocop and ignore the console warnings.
The text was updated successfully, but these errors were encountered: