Skip to content

Conversation

@Thomas-Shephard
Copy link
Contributor

Closes #33427

@dotnet-policy-service dotnet-policy-service bot added Community The pull request was submitted by a contributor who is not a Microsoft employee. VSCode labels Jan 4, 2026
@Thomas-Shephard Thomas-Shephard marked this pull request as ready for review January 5, 2026 00:06
@Thomas-Shephard Thomas-Shephard requested a review from a team as a code owner January 5, 2026 00:06

[UseExportProvider]
[Trait(Traits.Feature, Traits.Features.Completion)]
public sealed class ForeachCompletionTests : AbstractCSharpCompletionProviderTests
Copy link
Member

Choose a reason for hiding this comment

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

can you remove this, and instead add all tests to 'CSharpCompletionCommandHandlerTests".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Have moved these tests now :)

@Thomas-Shephard Thomas-Shephard force-pushed the feature/foreach-completion branch from cb61677 to 8219f29 Compare January 5, 2026 17:40
@Thomas-Shephard Thomas-Shephard force-pushed the feature/foreach-completion branch from 8219f29 to 843a757 Compare January 5, 2026 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-IDE Community The pull request was submitted by a contributor who is not a Microsoft employee. VSCode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Completion should preselect IEnumerable properties in foreach

2 participants