Fix up packaging for Extensions.ApiDescription.Server
Description
This change fixes up the package of dependencies for the dotnet-getdocument tool to enable changes that were made to support minimal APIs in .NET 7.
- Publish
GetDocument.Insiderdependency for all .NET Core TFMs - Use targeted referenced to shared runtime dependencies in
GetDocument.Insider - Add .NET 7 outputs to
ApiDescription.Serverpackage.
Fixes https://github.com/dotnet/aspnetcore/issues/44119
Customer Impact
Without this bug fix, customers are not able to leverage client generation for ASP.NET apps in Visual Studio for .NET 7 applications. There are no accessible workarounds for this bug.
Regression?
- [X] Yes
- [ ] No
Regression from .NET 7 RC 1. The bug is present in .NET 7 RC 2 and this PR is targeting .NET 7 GA.
Risk
- [ ] High
- [ ] Medium
- [X] Low
Change is localized to Extensions.ApiDescription.Server.
Verification
- [X] Manual (required)
Manual validation was completed by adding a reference to the local package output directory aspnetcore\artifacts\packages\Debug\Shipping and using the 7.0.0-dev version of the Microsoft.Extensions.ApiDescription.Server package in a sample project.
<Project Sdk="Microsoft.NET.Sdk.Web">
<PropertyGroup>
<TargetFramework>net7.0</TargetFramework>
<Nullable>enable</Nullable>
<ImplicitUsings>enable</ImplicitUsings>
<OpenApiGenerateDocumentsOnBuild>true</OpenApiGenerateDocumentsOnBuild>
</PropertyGroup>
<ItemGroup>
<PackageReference Include="Swashbuckle.AspNetCore" Version="6.4.0" />
<PackageReference Include="Microsoft.Extensions.ApiDescription.Server" Version="7.0.0-dev" PrivateAssets="all" />
<PackageReference Include="Microsoft.AspNetCore.OpenApi" Version="7.0.0-*" PrivateAssets="all" />
</ItemGroup>
</Project>
- [ ] Automated
Packaging changes reviewed?
- [X] Yes
- [ ] No
- [ ] N/A
Hi @captainsafia. Please make sure you've updated the PR description to use the Shiproom Template. Also, make sure this PR is not marked as a draft and is ready-to-merge.
To learn more about how to prepare a servicing PR click here.
Closing in favor of automerge from RC2.