-
Notifications
You must be signed in to change notification settings - Fork 116
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
bigdecimal no longer part of the default gems warning #219
Comments
I might need some help with this, because I am new with maintaining packages. I noticed just now that I had not run However I see you are installing it from nix. To update timetrap on nix, does someone need to make a PR and change the Gemfile.lock (nixpkgs). Are there nix maintainers that do this for this package? If you know, please let me know. Otherwise I would recommend just installing directly from rubygems |
Hmm, I fixed it in my fork of nixpkgs like so: jerith666/nixpkgs@61bcbed. I meant to open a PR to upstream that, but it looks like I never did. And maybe that's good, because it sounds like you're saying that isn't the right fix! :) Nixpkgs is currently on Line 28 in 95fcf74
If such a fix is made here and a new version released, I can take care of updating nixpkgs to use that new version. |
I have started getting warnings such as below when executing timetrap commands using version 1.15.2:
lib/ruby/gems/3.3.0/gems/sequel-5.30.0/lib/sequel.rb:3: warning: bigdecimal was loaded from the standard library, but will no longer be part of the default gems starting from Ruby 3.4.0. You can add bigdecimal to your Gemfile or gemspec to silence this warning.
I see from looking at the sequel code that this was fixed there in jeremyevans/sequel@c6a0178 and so should be in versions from 5.72.0 and later of sequel. Timetrap currently depends on version sequel 5.30.0 so can this be updated.
The text was updated successfully, but these errors were encountered: