Add a series of changes to make devcontainer experiences nicer for this repo #49867
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR enhances the development container experience for the repository by adding build automation, IDE configuration, and debugging support. The changes streamline the setup process and improve developer productivity when working in devcontainers.
- Automates the build process during container initialization and configures the IDE to use the locally built SDK
- Adds VS Code launch configurations for debugging the redist CLI with proper environment variables
- Disables automatic test discovery to improve performance and adds MCP server configuration
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
.devcontainer/scripts/post-creation.sh |
Adds build automation and SDK environment setup during container initialization |
.vscode/launch.json |
Provides debugging configurations for the .NET Core redist CLI |
.vscode/settings.json |
Disables automatic test discovery for better performance |
.vscode/mcp.json |
Configures MCP server for Microsoft documentation integration |
| "program": "${workspaceFolder}/artifacts/bin/redist/Debug/dotnet/sdk/10.0.100-dev/dotnet.dll", | ||
| "args": [ | ||
| ], | ||
| "env": { | ||
| "MSBuildExtensionsPath": "${workspaceFolder}/artifacts/bin/redist/Debug/dotnet/sdk/10.0.100-dev", | ||
| "MSBuildSDKsPath": "${workspaceFolder}/artifacts/bin/redist/Debug/dotnet/sdk/10.0.100-dev/Sdks", |
There was a problem hiding this comment.
So, these paths will break at each major version update?
There was a problem hiding this comment.
quite possibly, but there's not a great way to make this data-driven that I can see. normally you'd have devkit or something generate this for you, but since our setup is so particular their mechanisms don't work for us
There was a problem hiding this comment.
Yes, this will have to be updated with each major version.
There was a problem hiding this comment.
We can create a tasks.json with a preLaunchTask that uses the .NET Install Tool API dotnet.recommendedVersion to get the latest version of .NET and prepopulate the hard-coded value
…e SDK repo. * make devcontainer init script build the repo * source the sdk-build-env script so that the IDE experience uses the local SDK * setup simple launch configurations for debugging the redist CLI * disable the CDK test explorer auto-discovery because it takes _forever_ - trigger it once and then don't change anything instead
Co-authored-by: Copilot <[email protected]>
baronfel
force-pushed
the
codespaces-tweaks
branch
from
ba84905 to
819eab5
Compare
instead