-
Notifications
You must be signed in to change notification settings - Fork 145
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 requirement in lib/derailed_benchamrks/stats_from_dir.rb
#238
Conversation
as mentioned in the issue zombocom#237 the `ruby-statistics` gem updated their namespce of their gem from `statistics` to `ruby-statiscs` estebanz01/ruby-statistics@baadc9f and estebanz01/ruby-statistics@00312b9 this commit is addressing the change
Wont this break for those running older version of ruby-statistics? I would assume there would have to be a version checker when requiring or update the Gemfile to require v4 of ruby-statistics? |
@espen makes a good point - this PR needs to also bump |
yes, sorry when I opened this PR I was in firefighter mode and wanted to fix my deployment CI, when I have a free moment today I'll fix that |
@espen and @pcai done, my only concern is that I tried to run the test to check that this update didn't screw up anything but they crash even on the original main with:
So I don't think this is related to my PR, but it is still concerning |
Hmm the rails version constraint looks a bit unorthodox. I would expect it to read < 7 if the intent was 6.x or lower or < 8 if the intent was 7.x or lower. If im reading correctly, it allows the initial 7.0.0 but none of the patches, which might be the problem? |
@deepakmahakale I see you helped maintain this repo recently, any thoughts on above? |
I investigated the test failures and they are mostly* unrelated. I'm going to repair the bitrot in a separate PR to get main green again. Thanks for your contribution. *We may need to do some branching to maintain support for legacy ruby versions |
as mentioned in the issue
#237 by @espen, the
ruby-statistics
gem updated the namespace of their gem fromstatistics
toruby-statistics
estebanz01/ruby-statistics@baadc9f and
estebanz01/ruby-statistics@00312b9 this commit is addressing the change