Skip to content

Conversation

@jasonmalinowski
Copy link
Member

This rename was applied to all the common settings (tab/spaces, etc) and is expected of us as well.

This rename was applied to all the common settings (tab/spaces, etc)
and is expected of us as well.
@jasonmalinowski jasonmalinowski self-assigned this Sep 16, 2025
@jasonmalinowski jasonmalinowski requested a review from a team as a code owner September 16, 2025 20:54
Add(CompletionOptionsStorage.ShowItemsFromUnimportedNamespaces, "textEditor.csharp.intellisense.showCompletionItemsFromUnimportedNamespaces").
Add(CompletionViewOptionsStorage.EnableArgumentCompletionSnippets, "textEditor.csharp.intellisense.enableArgumentCompletionSnippets").
Add(CompletionOptionsStorage.ShowNewSnippetExperienceUserOption, "textEditor.csharp.intellisense.showNewSnippetExperience");
Add(CompletionOptionsStorage.TriggerOnTypingLetters, "languages.csharp.intellisense.triggerCompletionOnTypingLetters").
Copy link
Member Author

Choose a reason for hiding this comment

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

I'm making a note to dig into what these tests are adding here. It appears that for the most part it's just asserting the contents of the JSON match the contents of this, which doesn't actually catch bugs -- it just means you get to make every change twice. There is a bit of useful stuff though in it around checking resource strings, so I don't want to entirely delete it now.

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