-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Update decouple-vs-and-net-sdk.md #48844
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Putting the actual package name in the documentation for quicker reference.
|
@baronfel PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR updates the documentation to display the correct package name for the .NET Framework compiler in SDK projects.
- Updated the package name in the "FrameworkPackage" section for clarity.
| > - `Core`: use the compiler that comes with the .NET SDK | ||
| > - `Framework`: use the compiler that comes with .NET Framework MSBuild | ||
| > - `FrameworkPackage`: download package with .NET Framework compiler corresponding to the .NET SDK version | ||
| > - `FrameworkPackage`: download the Microsoft.Net.Sdk.Compilers.Toolse package which contains the .NET Framework compiler corresponding to the .NET SDK version |
Copilot
AI
May 7, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The package name 'Microsoft.Net.Sdk.Compilers.Toolse' appears to include a typo. Verify that the intended name is correct, such as 'Microsoft.Net.Sdk.Compilers.Tools'.
| > - `FrameworkPackage`: download the Microsoft.Net.Sdk.Compilers.Toolse package which contains the .NET Framework compiler corresponding to the .NET SDK version | |
| > - `FrameworkPackage`: download the Microsoft.Net.Sdk.Compilers.Tools package which contains the .NET Framework compiler corresponding to the .NET SDK version |
|
@baronfel this PR is stuck and it's just a doc change. Can you Yoda the PR? |
|
/ba-g |
|
/ba-g helix is stuck? |
Putting the actual package name in the documentation for quicker reference.