-
Notifications
You must be signed in to change notification settings - Fork 877
feat: Add .slnx solution format metadata generation support
#10540
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
|
Thank you @filzrev |
|
|
||
| <!-- .slnx solution format is supported Microsoft.Build 17.13.9 or later. --> | ||
| <ItemGroup> | ||
| <PackageVersion Include="Microsoft.Build" Version="[17.11.4]" Condition="'$(TargetFramework)' == 'net8.0'"/> |
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.
Is it possible to use 17.13.9 for net8.0 so that it can also be supported for net8.0?
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.
Microsoft.Build 17.13.9 package is published as .NET 9 target.
https://www.nuget.org/packages/Microsoft.Build/17.13.9#supportedframeworks-body-tab
So It's not possible to reference this version from .NET 8 applications.
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.
👍 thanks for the info
This PR add support metadata generation for
.slnxsolution format.Note:
It requires
Microsoft.Buildpackage version17.13.9or later. (Requires .NET 9).So it works only when running docfx with .NET 9.
Test
It's manually tested by following steps.
samples\seed\dotnet\solution\BuildFromSolution.slnto.slnxformat by using Visual Studio.2 Running
docfx metadatacommand with generated.slnxsolution file.