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

Fix #1664 for Hash#compare_by_identity and Hash#store #1665

Closed
wants to merge 1 commit into from

Conversation

godfat
Copy link
Contributor

@godfat godfat commented Apr 26, 2014

#1664

Test in RubySpec:
https://github.com/rubyspec/rubyspec/pull/274

Sorry that I didn't check if this works. Let's see travis :P

@lucasallan
Copy link
Member

Hey @godfat
Could you send this pull request to jruby-1_7 branch? This needs to go into the next 1.7.x patch release.

We gonna merge it into master later.
Thanks

@godfat
Copy link
Contributor Author

godfat commented Apr 26, 2014

Hi,

I probably cannot change that, therefore I created a new one:
(also removed the whitespaces change given this chance)
#1666

Thanks for merging!

@lucasallan
Copy link
Member

We gonna review and test #1666 and give you a feedback soon. I'm gonna close this pull request since we don't need it anymore.
Thanks for the contribution!

@lucasallan lucasallan closed this Apr 26, 2014
@godfat
Copy link
Contributor Author

godfat commented Apr 26, 2014

Thanks for reviewing and testing!

@enebo enebo added this to the Invalid or Duplicate milestone Jun 24, 2014
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.

None yet

3 participants