Skip to content

Use partial semantic model in import completion#51327

Merged
genlu merged 8 commits intodotnet:masterfrom
genlu:partialModel
Feb 23, 2021
Merged

Use partial semantic model in import completion#51327
genlu merged 8 commits intodotnet:masterfrom
genlu:partialModel

Conversation

@genlu
Copy link
Member

@genlu genlu commented Feb 19, 2021

Fixes AB#1275634

@genlu genlu requested a review from a team as a code owner February 19, 2021 00:53
@ghost ghost added the Area-IDE label Feb 19, 2021
@sharwell sharwell marked this pull request as draft February 19, 2021 17:05
.WithChangedOption(CompletionOptions.SnippetsBehavior, document.Project.Language, SnippetsRule.NeverInclude)
.WithChangedOption(CompletionOptions.ShowItemsFromUnimportedNamespaces, document.Project.Language, false);
.WithChangedOption(CompletionOptions.ShowItemsFromUnimportedNamespaces, document.Project.Language, false)
.WithChangedOption(CompletionServiceOptions.IsExpandedCompletion, false);
Copy link
Contributor

Choose a reason for hiding this comment

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

doc.

Copy link
Member Author

Choose a reason for hiding this comment

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

Will add in a follow up PR

@CyrusNajmabadi
Copy link
Contributor

Use partial semantic model in import completion

can we move to this for all completion providers?

@genlu genlu marked this pull request as ready for review February 22, 2021 19:25
@genlu
Copy link
Member Author

genlu commented Feb 22, 2021

can we move to this for all completion providers?

I can do this in a follow up PR.

@genlu genlu merged commit 4382a2c into dotnet:master Feb 23, 2021
@ghost ghost added this to the Next milestone Feb 23, 2021
@genlu genlu deleted the partialModel branch February 23, 2021 00:30
@allisonchou allisonchou modified the milestones: Next, 16.10.P2 Mar 29, 2021
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