-
Notifications
You must be signed in to change notification settings - Fork 73
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
Maintenance #76
Comments
Agree. Need to let someone else help you with this repo so we can have a stable BigDecimal in rust. |
I'm sorry I haven't been paying attention to this project. The efficiency is quite terrible and last December I had actually started reimplementing everything using I figured the reimplementation (which kind of has to happen, if we want good performance) would be the place to implement the useful things like context which would answer many of the issues people have, but obviously it's taking too long. I'll go back to focusing on maintaining the version as-is, and I'll check into GitHub more often. Again, sorry if this has caused any inconvenience. Also, I'd like to everyone interested to discuss implementation details on the project gitter. Though, since nobody is using it, I wonder if there's a more popular chat solution that rustaceans would prefer? |
I would say BigDecimal is a popular and valuable library in the Rust ecosystem. I would be proud of that and see it as worth maintaining...already have good traction, keep it going. Deciding to re-write a project is a tough decision I know, and it takes a lot of time and effort. I do suggest chipping away at a new version if performance is really that bad, but in the mean time, put a bit of effort into the old code. I would suggest going through all these issues and fixing things up to make a good stable 1.0 of this older slower code. It's also nice that people have already fixed your code with PR's waiting for you to merge. Once 1.0 is stable I would casually work on a "next" branch for 2.0 re-write / performance booster. 1.0 could be feature frozen forever, although real bugs should always be fixed, even in frozen releases, they are important to give a bit of energy too. Not sure about gitter or best place, but for me personally, I like all things to be on one place, right here on github. Easier for other people too. Github added a new "discussions" feature you can enable in the repo settings. Good for separating actual issues vs discussions like these. |
It looks like this crate is hardly maintained. Many of the open issues, most of them many months old, haven't received any comments, and four PRs (the latest one from Nov 2020) are also sitting there without any feedback.
@akubera Do you intend to more actively maintain this crate soon? If not, would you be open to
Since this crate is a dependency (though optional) of both diesel and SQLx, it probably won't be that hard to find people willing to help with maintenance.
The text was updated successfully, but these errors were encountered: