Skip to content

Set context flag for when C++ language server is activated#14382

Merged
DavidARaygoza merged 1 commit intomainfrom
dev/davidraygoza/SetContextForCppLanguageServer
Apr 16, 2026
Merged

Set context flag for when C++ language server is activated#14382
DavidARaygoza merged 1 commit intomainfrom
dev/davidraygoza/SetContextForCppLanguageServer

Conversation

@DavidARaygoza
Copy link
Copy Markdown
Member

The primary purpose of this is for this flag to be consumed by the C++ devtools extensions. This way the C++ specific Copilot tools will only appear in tools list and register when the C++ language server is actually activated.

bobbrow
bobbrow previously approved these changes Apr 15, 2026
Comment thread Extension/src/LanguageServer/extension.ts Outdated
The primary purpose of this is for this flag to be consumed by the C++ devtools extensions. This way the C++ specific Copilot tools will only appear in tools list and register when the C++ language server is actually activated.
@DavidARaygoza DavidARaygoza force-pushed the dev/davidraygoza/SetContextForCppLanguageServer branch from 5b2dd9b to 480f570 Compare April 16, 2026 00:19
@DavidARaygoza DavidARaygoza requested a review from a team as a code owner April 16, 2026 00:19
Copy link
Copy Markdown
Contributor

@sean-mcmanus sean-mcmanus left a comment

Choose a reason for hiding this comment

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

Should it be set to false if the language server is disabled?

@sean-mcmanus
Copy link
Copy Markdown
Contributor

Should it be set to false if the language server is disabled?

Oh, it appears we require a Reload Window in that case so it's not an issue.

@DavidARaygoza DavidARaygoza merged commit 7665e11 into main Apr 16, 2026
6 checks passed
@DavidARaygoza DavidARaygoza deleted the dev/davidraygoza/SetContextForCppLanguageServer branch April 16, 2026 00:24
@github-project-automation github-project-automation bot moved this from Pull Request to Done in cpptools Apr 16, 2026
@sean-mcmanus
Copy link
Copy Markdown
Contributor

Well, actually, I don't know what happens if the C/C++ is disabled via the Extension window....but I can try that out later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants