-
Notifications
You must be signed in to change notification settings - Fork 7
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
Error: uninitialized constant Bignum (NameError) #15
Comments
Unfortunately, I'm not sure what can be done on my end. Someone needs to take ownership of edn-ruby as it seems relevance (cognitect) have completely dropped supporting it. I'm not sure how easy that is on the rubygems side, though? Alternatively, an option might be to inherit edn-ruby into the edn_turbo source and turn it into its own gem? I have no idea how that may break things for people upgrading however. I also don't do much work on Ruby these days but, if several folks seem interested in doing something like this, I'd be happy to help with the migration and release. |
If https://github.com/relevance/edn-ruby can no longer be maintained, then I suppose a fork may be needed and a new Ruby Gem built. As you suggest inheriting edn-ruby into the edn_turbo source may also be a path forward. I've sent an email to @russolsen hoping he may be able to weigh in on this. If the edn-ruby repo can be updated, I'm guessing relevance/edn-ruby#47 may be the fix. ping @caleb |
A drop-in replacement has been created. The new gem is The only update needed on your part is in
To:
You may also want to update the "Dependencies" section in See: Thank you for your help! |
I'm getting a 404 on that repo |
Thanks for catching that. Repo has been made publicly visible. |
Related to: relevance/edn-ruby#39
The
edn
gem dependency relies on the RubyBignum
class. This has been deprecated for some time and has now been removed from Ruby 3.2.Loading edn_turbo now results in an error. Is there any possible workaround?
The text was updated successfully, but these errors were encountered: