-
Notifications
You must be signed in to change notification settings - Fork 868
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
feat: Add .slnx
solution format metadata generation support
#10540
Conversation
Thank you @filzrev |
@@ -25,6 +25,12 @@ | |||
<PackageVersion Include="YamlDotNet" Version="15.3.0" /> | |||
</ItemGroup> | |||
|
|||
<!-- .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
.slnx
solution format.Note:
It requires
Microsoft.Build
package version17.13.9
or 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.sln
to.slnx
format by using Visual Studio.2 Running
docfx metadata
command with generated.slnx
solution file.