Skip to content

Add ExternalAccess for INavigateToSearcherService#63386

Merged
333fred merged 2 commits intodotnet:mainfrom
333fred:inavigatetosearchservice-ea
Aug 13, 2022
Merged

Add ExternalAccess for INavigateToSearcherService#63386
333fred merged 2 commits intodotnet:mainfrom
333fred:inavigatetosearchservice-ea

Conversation

@333fred
Copy link
Copy Markdown
Member

@333fred 333fred commented Aug 13, 2022

Because of #63375, O# cannot currently use SymbolFinder for finding all symbols for a pattern in the workspace. As a workaround, expose INavigateToSearcherService via EA to unblock the workspace symbol service.

Because of dotnet#63375, O# cannot currently use SymbolFinder for finding all symbols for a pattern in the workspace. As a workaround, expose INavigateToSearcherService via EA to unblock the workspace symbol service.
@ghost ghost added the Area-IDE label Aug 13, 2022
Comment thread src/Tools/ExternalAccess/OmniSharp/NavigateTo/OmniSharpNavigateToSearchService.cs Outdated
Copy link
Copy Markdown
Contributor

@CyrusNajmabadi CyrusNajmabadi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM except for: https://github.com/dotnet/roslyn/pull/63386/files#r945190376

I think you hopefully can get this working. But not hugely problematic if you don't i don't think.

@333fred 333fred marked this pull request as ready for review August 13, 2022 21:21
@333fred 333fred requested a review from JoeRobich as a code owner August 13, 2022 21:21
@333fred 333fred enabled auto-merge (squash) August 13, 2022 21:21
@333fred 333fred merged commit 1d66f3a into dotnet:main Aug 13, 2022
@ghost ghost added this to the Next milestone Aug 13, 2022
@333fred 333fred deleted the inavigatetosearchservice-ea branch August 13, 2022 22:57
@dibarbet dibarbet modified the milestones: Next, 17.4 P2 Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants