-
-
Notifications
You must be signed in to change notification settings - Fork 184
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
Using outdated shoulda-matchers version #275
Comments
As infrequently as this gem is updated, I don't think removing the version restrictions would be a good idea, because it goes against expectations around gems and backward compatibility. Specifically, it means that two developers who ran |
Oh, all my projects use the Gemfile.lock to ensure that this doesn't happen. I didn't know that there are different setups where this could be a problem. So if you are happy to update, that's great. Feel free to ignore my suggestions. The current issue is just that it hasn't been updated and that caused incompatibilities in our code. |
I also ran into this issue as I thought there was a bug in |
Yeah, this makes no sense. It used to be the case that bundler advised gem authors to not commit Our (legacy) Gemfile (these lines haven't changed in 6 years):
When running It downgrades from This is worse than having a vague version constraint that would allow bundler to "do the right thing", I.e. get the latest versions of everything it can. I'm going to remove |
Sorry folks, we've neglected this gem it seems. I'll work on preparing a new release that upgrades this gem to the current version of |
@mcmire thank you for your work. Is there any progress, yet? Do you need support? |
@mcmire Looks like you're not at Thoughtbot anymore. Do you know if someone else will be picking this up, or are you still involved? |
Thanks for the ping, everyone, and sorry for the lack of response here. Things have been a bit busy lately. I dusted this gem off again and there were some things that were pretty out of date. For one, I couldn't even run the tests anymore because it was relying on a version of Ruby that doesn't compile on recent versions of macOS anymore. Lately we've been making some upgrades to So, I just released 5.0.0.rc1 which includes these updates and, of course, bumps the dependency on |
I'm working on a project that sets gem authors up with a docker image for each version of ruby they'd like to run their gems tests against. You can run it locally or in ci -- I'm using circleci. Let me know if you think that would be helpful. Otherwise, I appreciate your work! |
Hi @mcmire and thanks for releasing Since |
@schadenfred Please do say more about this. I have a project of RuboCop wrappers (called |
Broad strokes -- The project is called RoRo. You can think of it as a framework for writing configuration backstories and then sharing them. One of the backstories I wrote is for gem authors, but I think I'm the only one who has used it. I haven't had to test any of my gems against ruby 1.8, but I looked around and found some alpine docker images with it, as well as some bigger ubuntu based ones. image size ma may not be such a big deal depending on what you'd want to use for ci. Do you have experience with CircleCI? Do you have a preference for something else? Which of your repos would you want to set up first? |
I just noticed that we can update shoulda-matchers after removing this gem. There seem to be some incompatibilities between shoulda-matchers 4 and newer Rails versions. Since there's not much time for maintaining this gem, I'm wondering about these options:
The text was updated successfully, but these errors were encountered: