-
Notifications
You must be signed in to change notification settings - Fork 215
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
Fix deprecated method File.exists? to File.exist? #159
Conversation
@knowtheory @jashkenas Any chance we get this merged? |
Hi, I just ran into the same issue when trying to upgrade to Ruby 3.2. It would be great, if this could get merged. It should also not break compatibility to < 3.2-versions. |
@tuttiq any news about this topic? |
@tsotne-m (cc @tmaier) I ended up pointing the source for the gem (on my project's Gemfile) to my forked version: https://github.com/tuttiq/docsplit Not great, but I figured this repository is no longer being maintained 🤷♀️ I don't plan on maintaining my fork either (since I'm not working on that project anymore), so I recommend you maintain your own forks if you need this gem long term. |
@tuttiq Thanks a lot for response |
I'm just working on getting rid of Docsplit as well and it depends on your usecase, but in my case of using it to extract word processing documents, it looks like switching to something like libreconv (or just LibreOffice directly) to convert the document to PDF and then use pdf-reader to extract the text is the way to go. |
I consider to use Apache Tika in the future. Especially, a tika microservice. It has a simple REST API to extract text. See https://cwiki.apache.org/confluence/display/TIKA/TikaServer#TikaServer-GettheTextofaDocument |
Why this PR isn't merged? This is a common issue for all +Ruby 3.2 projects... |
❤️ |
have these changes been pushed to rubygems by any chance? as a workaround for our projects, we had to point the our gem to this repo's git directly. |
Fix for this issue: #158
Fixes compatibility with ruby 3.2.