Conversation
|
May I ask the rationale and/or vision behind this contribution? |
|
The C/C++ Dev Tools extension includes experiences that build on existing C/C++ extensions to improve the end-to-end C++ workflow in VS Code. GitHub Copilot is installed by default in VS Code, and our goal is to ensure C++ developers get consistent, high quality C++ experiences out of the box. To do so, the C/C++ DevTools extension adds workflow enhancements that rely on services implemented by our other C++ extensions. |
@sinemakinci1 Thanks for answer. |
|
@sinemakinci1 & @gcampbell-msft From a technical and policy standpoint, I consider this contribution quite unusual. Here, we have an extension acting as an extension pack. I would welcome a new extension pack or an enhancement to an existing one like ms-vscode.cpptools, which bundles ms-vscode.cmake-tools and the new ms-vscode.cpp-devtools. |
|
@VGRSTM Our goal was to provide the best default experience for our users, since this new extension is a companion to the CMake tools extension, which is why we chose this dependency pattern. If you decide this setup doesn't work for you, you can uninstall the C++ devtools extension at any point and your existing CMake and C/C++ tooling will continue to work as before. Thanks for raising this, feedback like this is helpful as we iterate on the technical approach and the way we communicate these changes. |
|
@sinemakinci1 & @gcampbell-msft I agree with you @VGRSTM, thanks for linking my issue here. This extension is already part of the C/C++ Extension Pack, so users who want the full C++ experience already get it. Adding it as an extension pack entry in CMake Tools means it gets silently installed on users who deliberately chose CMake Tools alone rather than the full pack. The marketplace desc saying "Enhanced development tools for C++" also undersells that it's a Copilot integration. You only find that out in the Overview. Users who don't use Copilot end up with an extra closed-source extension that does nothing for them. Would it be possible to keep this dependency only in the C/C++ Extension Pack where it already lives, or at the very least have it respect |
|
I've opened a PR to revert this change: #4800 |
This change addresses item #[[put issue number here to generate a link]]
This changes [[visible behavior/performance/documentation/etc.]]
The following changes are proposed:
The purpose of this change
Other Notes/Information