-
Notifications
You must be signed in to change notification settings - Fork 29.4k
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
Stabilize findTextInFiles API #59924
Comments
@andreamah I really like this feature for my next side project, and may I ask here how & when can this be stabilized? It's been five years with no critical issue happening on it (let me know if I missed sth). |
Some news: I'm finalizing this API with some other related APIs. You can see the status (plus a diagram) here :) |
The (mostly-finalized-but-still-under-discussion) new API shape for this is now available in this file. It will eventually have the |
Hi @andreamah I do plan to use this thank you. I manage the helix extension which is similar to vscode-vim, the downside with these extensions is they need to implement their own search (which is slow on large files) Do you know if there is a version for finding text in a single file? Or can this be used with some mode to only search the focused editor? I also hope to use the Regex options too like you can in the UI, plus only searching within selected text. |
Thanks for working on this - we'd like to implement search in a VSCode extension, and doing it from scratch is either really slow or really complicated. Any ETA on when this will be finalized? Thanks! |
Apologies on the late reply.
Probably 2+ months at this point. This API is tied into 3 others, and the TODOs are:
|
Thanks @andreamah I can't seem to see Regex support in this API, our extension would need that, is that something you plan on adding in future? The UI allows this so I don't see this being difficult to support at an API level. |
This API's types are shared with vscode/src/vscode-dts/vscode.proposed.textSearchProviderNew.d.ts Lines 30 to 36 in dfb96d1
The naming for all of these are temporary (so when finalized, they won't have |
Depends on
The text was updated successfully, but these errors were encountered: