Skip to content

Conversation

tanaka51
Copy link

@tanaka51 tanaka51 commented Sep 13, 2016

Hi,

I can't install this gem, jsonapi, on Ruby 2.4(preview1 and preview2) because Ruby 2.4 unifies Bignum and Fixnum to Integer and the json gem version 1.8 that this gem dependents on uses both classes. Now, supported version is released as 2.0.
To resolve this issue, I simply removed the json gem dependency.

Thanks.

@tanaka51 tanaka51 force-pushed the support-ruby-2-4 branch 2 times, most recently from 873c876 to bb2111c Compare September 13, 2016 18:31
@jasnow
Copy link

jasnow commented Sep 14, 2016

@tanaka51 - Does it work with preview2?

@tanaka51
Copy link
Author

@jasnow Yes, it works on my end. (Travis CI does not support preview2 yet...

@beauby beauby mentioned this pull request Sep 18, 2016
@beauby
Copy link
Owner

beauby commented Sep 18, 2016

Thanks for the PR @tanaka51 – I think the best way around all those issues is, indeed, to remove the dependency on the json gem (as it does not provide much value in this gem anyway). Let's take care of it in #33, and close this PR.

@beauby beauby closed this Sep 18, 2016
@tanaka51
Copy link
Author

tanaka51 commented Sep 21, 2016

@beauby Sorry for late response. I totally agree with your opinion. Thank you for reviewing my PR :)

@tanaka51 tanaka51 deleted the support-ruby-2-4 branch September 21, 2016 17:08
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.

3 participants