feat: add xlsx and ods general fallback support #241
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Background
I attempted to use Custom Adapter suggested in #36 (comment) and configure like this
However, it did not show the match and always show unknown error like
Even I just put command line (
binary
andargs
in the config) to terminal and run successfully without no issue.From reading code, just found this line
ripgrep-all/src/adapters/zip.rs
Line 10 in e207c12
ods
andxlsx
is zip archived and can be searched from extraction.Purpose
Just provide a fallback general availability to search zipped document that not supported by pandoc or other tools etc. If custom adapter is configured, it will use custom adapter rather than this config.
.epub
can be considered as a zipped package if following this idea.Further
Basically it's just adhoc tweak, but hopefully provide a temporary mitigation for the pains from users. If we have any better perm/adboc solution, I'm glad to help on that.