Skip to content

Avoid Fixnum deprecation warning on Ruby trunk #660

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

Merged
merged 1 commit into from
Dec 20, 2016

Conversation

utilum
Copy link
Contributor

@utilum utilum commented Dec 17, 2016

No description provided.

@irobayna
Copy link

Any chance we can merge this item?

@badboy
Copy link
Contributor

badboy commented Dec 20, 2016

Let me grab 2.4-rc1 and see for myself. Otherwise I don't see why not.

@badboy badboy merged commit 37ab904 into redis:master Dec 20, 2016
@badboy
Copy link
Contributor

badboy commented Dec 20, 2016

Thanks @utilum for the patch, thanks @irobayna for the reminder.

@utilum utilum deleted the avoid_fixnum_deprecation branch December 20, 2016 13:18
@matthewrudy
Copy link

Any chance of getting a release of this?
Would be great to have ruby 2.4.0 running without any deprecations

@badboy
Copy link
Contributor

badboy commented Dec 27, 2016

Hm, we should probably get 4.0 out of the door.

@manuelmeurer
Copy link

Would love to see v3.3.3 with this change released! 😄

@vincentwoo
Copy link

Just to echo the sentiments here in case this got forgotten in the holiday shuffle, I think a tiny patch release just so we can silence the deprecation warnings would be great!

@george-carlin
Copy link

Just so everyone knows, it looks like 3.3.3 contains the above commit. It's not reflected in the changelog for 3.3.3, but I just submitted a PR to add it ;)

@matthewrudy
Copy link

@georgemillo yep
I switched to 3.3.3 a while ago,
works for me!

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.

7 participants