-
Notifications
You must be signed in to change notification settings - Fork 30.6k
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
Can't search in workspace that includes folder using a FileSystemProvider #94928
Comments
I as author of the extension which implements file system provider can't implement search provider since it's only in proposed API. |
it is.but right now this is make issue in v1.44.0. Is that expected? this is not duplicate since this don't ask for feature but rater bug related to this feature. |
is that expected that users of my extension have to between choose to use file search or work with external file system? because feature is in proposed stage but causes issues in stable release PS. please note; v1.43.0 works fine PPS:
means issue is not related to the extension. I'm correct? |
@roblourens please reopen issue since it's not fixed and not duplicate |
@roblourens or @RMacfarlane please suggest how can I fix the issue |
I'm pretty sure this didn't work in old versions either, the API has never been stable. |
let's start from scratch. Extension don't use search provider API ( I don't need proposed API. I need search working again. |
@roblourens by the way: |
@roblourens Maybe this commit is the problem? 0e219d4 Would this cause search to wait indefinitely? |
Is this a multiroot workspace where one folder is a normal folder on disk and the second is from the separate scheme? I guess I see the issue then. If that's not it then I guess I don't understand what was being searched before, please explain. |
correct, workspace has few folders in it with one "virutal" for {
"folders": [
{
"path": "ecom-project"
},
{
"name": "Sandbox - FileSystem",
"uri": "ccfs://current-sandbox"
}
],
"settings": {}
} |
@roblourens this 1.44 regression is likely to affect users of one of my extensions too. Please make the fix a candidate for the 1.44.1 recovery release. |
Yeah I see it. That change was a mistake, I agree that it should be a candidate. Thanks for finding the commit @RMacfarlane, I think the fix is just to revert that and work around this in the smoke test. |
Hi, I'm getting the following log after the 1.44.0 update whenever I CTRL+P (search hangs and does not show results):
disinstalling, reinstalling gitlens or disabling extensions was of no help. Recreating the workspace fixed it.
|
I'm having the same problem as @Feder1co5oave . I even uninstalled GitLens but the problem persists. Search is totally broken now. |
That said, how do I manually remove the GitLens search provider? |
This is fixed in Insiders but not stable yet. |
Verification steps:
|
Steps to Reproduce:
Does this issue occur when all extensions are disabled?: Yes
The text was updated successfully, but these errors were encountered: