-
Notifications
You must be signed in to change notification settings - Fork 55
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 float issue with object space dump #146
Fix float issue with object space dump #146
Conversation
.tool-versions
Outdated
@@ -1 +1 @@ | |||
ruby 2.5.9 | |||
ruby 2.7.1 |
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.
Thanks for the change! Mind putting this in another PR? This seems like something someone may want to track down later and I don't want it to get lost in this one.
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.
Actually, take that back — since we still support Ruby 2.5 as the minimum tested version, we probably want to leave this alone :)
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.
I reverted it in 2450859
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.
I will fix this error in the test suite
NoMethodError:
undefined method `try' for #<Hash>
Did you mean? pry
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.
I think 55d9dc2 should fix that issue
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.
The remainder that are failing are probably due to an issue with ruby version + minitest version issues, I pushed up #148 to try and fix the issue
This reverts commit a663893.
I merged |
Done and merged! |
A fix for #144 and align
.tool-version
with.ruby-version