Skip to content

Commit 58e7bc7

Browse files
Copilotjoperezr
andauthored
Add Workstream, Stage, and PackageValidationBaselineVersion metadata to ServiceDiscovery libraries (#6919)
* Initial plan * Add Workstream and Stage metadata to ServiceDiscovery libraries Co-authored-by: joperezr <[email protected]> * Add PackageValidationBaselineVersion to ServiceDiscovery projects Co-authored-by: joperezr <[email protected]> * Only set default PackageValidationBaselineVersion when not explicitly provided * Add StrongNameKeyId and test coverage metadata to ServiceDiscovery projects Replace PackageIconFullPath with StrongNameKeyId and add MinCodeCoverage/MinMutationScore properties in ServiceDiscovery, ServiceDiscovery.Abstractions, ServiceDiscovery.Dns, and ServiceDiscovery.Yarp .csproj files. Also add empty .json placeholder files for ServiceDiscovery and ServiceDiscovery.Abstractions. * Set StrongNameKeyId to Open in ServiceDiscovery test projects --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: joperezr <[email protected]> Co-authored-by: Jose Perez Rodriguez <[email protected]>
1 parent 4e17769 commit 58e7bc7

File tree

13 files changed

+41
-5
lines changed

13 files changed

+41
-5
lines changed

eng/MSBuild/Packaging.targets

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
<!-- https://learn.microsoft.com/dotnet/fundamentals/package-validation/overview -->
3838
<EnablePackageValidation>true</EnablePackageValidation>
3939

40-
<PackageValidationBaselineVersion Condition=" '$(Stage)' == 'normal' ">9.8.0</PackageValidationBaselineVersion>
40+
<PackageValidationBaselineVersion Condition="'$(Stage)' == 'normal' and '$(PackageValidationBaselineVersion)' == ''">9.8.0</PackageValidationBaselineVersion>
4141
</PropertyGroup>
4242

4343
<!-- Verify that the minimum supported TFM is actually used. -->

src/Libraries/Microsoft.Extensions.ServiceDiscovery.Abstractions/Microsoft.Extensions.ServiceDiscovery.Abstractions.csproj

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,21 @@
44
<TargetFrameworks>$(TargetFrameworks);netstandard2.0</TargetFrameworks>
55
<IsPackable>true</IsPackable>
66
<Description>Provides abstractions for service discovery. Interfaces defined in this package are implemented in Microsoft.Extensions.ServiceDiscovery and other service discovery packages.</Description>
7-
<PackageIconFullPath>$(DefaultDotnetIconFullPath)</PackageIconFullPath>
7+
<StrongNameKeyId>Open</StrongNameKeyId>
88
<RootNamespace>Microsoft.Extensions.ServiceDiscovery</RootNamespace>
99
<!-- https://github.com/dotnet/extensions/issues/6871 tracks enabling these -->
1010
<NoWarn>$(NoWarn);S1144;CA1002;S2365;SA1642;IDE0040;CA1307;EA0009;LA0003</NoWarn>
1111
<ImplicitUsings>enable</ImplicitUsings>
1212
</PropertyGroup>
1313

14+
<PropertyGroup>
15+
<Workstream>ServiceDiscovery</Workstream>
16+
<Stage>normal</Stage>
17+
<PackageValidationBaselineVersion>9.5.1</PackageValidationBaselineVersion>
18+
<MinCodeCoverage>75</MinCodeCoverage>
19+
<MinMutationScore>75</MinMutationScore>
20+
</PropertyGroup>
21+
1422
<ItemGroup>
1523
<PackageReference Include="Microsoft.Extensions.Configuration.Abstractions" />
1624
<PackageReference Include="Microsoft.Extensions.DependencyInjection.Abstractions" />

src/Libraries/Microsoft.Extensions.ServiceDiscovery.Abstractions/Microsoft.Extensions.ServiceDiscovery.Abstractions.json

Whitespace-only changes.

src/Libraries/Microsoft.Extensions.ServiceDiscovery.Dns/Microsoft.Extensions.ServiceDiscovery.Dns.csproj

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,21 @@
44
<TargetFrameworks>$(NetCoreTargetFrameworks)</TargetFrameworks>
55
<IsPackable>true</IsPackable>
66
<Description>Provides extensions to HttpClient to resolve well-known hostnames to concrete endpoints based on DNS records. Useful for service resolution in orchestrators such as Kubernetes.</Description>
7-
<PackageIconFullPath>$(DefaultDotnetIconFullPath)</PackageIconFullPath>
7+
<StrongNameKeyId>Open</StrongNameKeyId>
88
<!-- https://github.com/dotnet/extensions/issues/6871 tracks enabling these -->
99
<NoWarn>$(NoWarn);IDE0018;IDE0025;IDE0032;IDE0040;IDE0058;IDE0250;IDE0251;IDE1006;CA1304;CA1307;CA1309;CA1310;CA1849;CA2000;CA2213;CA2217;S125;S1135;S1226;S2344;S3626;S4022;SA1108;SA1120;SA1128;SA1129;SA1204;SA1205;SA1214;SA1400;SA1405;SA1408;SA1515;SA1600;SA1629;SA1642;SA1649;EA0001;EA0009;EA0014;LA0001;LA0003;LA0008;VSTHRD200</NoWarn>
1010
<ImplicitUsings>enable</ImplicitUsings>
1111
<DisableMicrosoftExtensionsLoggingSourceGenerator>false</DisableMicrosoftExtensionsLoggingSourceGenerator>
1212
</PropertyGroup>
1313

14+
<PropertyGroup>
15+
<Workstream>ServiceDiscovery</Workstream>
16+
<Stage>normal</Stage>
17+
<PackageValidationBaselineVersion>9.5.1</PackageValidationBaselineVersion>
18+
<MinCodeCoverage>75</MinCodeCoverage>
19+
<MinMutationScore>75</MinMutationScore>
20+
</PropertyGroup>
21+
1422
<ItemGroup>
1523
<PackageReference Include="DnsClient" />
1624
<PackageReference Include="Microsoft.Extensions.Configuration.Abstractions" />

src/Libraries/Microsoft.Extensions.ServiceDiscovery.Dns/Microsoft.Extensions.ServiceDiscovery.Dns.json

Whitespace-only changes.

src/Libraries/Microsoft.Extensions.ServiceDiscovery.Yarp/Microsoft.Extensions.ServiceDiscovery.Yarp.csproj

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,20 @@
66
<Nullable>enable</Nullable>
77
<IsPackable>true</IsPackable>
88
<Description>Provides extensions for service discovery for the YARP reverse proxy.</Description>
9-
<PackageIconFullPath>$(DefaultDotnetIconFullPath)</PackageIconFullPath>
9+
<StrongNameKeyId>Open</StrongNameKeyId>
1010
<!-- https://github.com/dotnet/extensions/issues/6871 tracks enabling these -->
1111
<NoWarn>$(NoWarn);IDE0018;IDE0025;IDE0032;IDE0040;IDE0058;IDE0250;IDE0251;IDE1006;CA1304;CA1307;CA1309;CA1310;CA1849;CA2000;CA2213;CA2217;S125;S1135;S1226;S2344;S2692;S3626;S4022;SA1108;SA1120;SA1128;SA1129;SA1204;SA1205;SA1214;SA1400;SA1405;SA1408;SA1414;SA1515;SA1600;SA1615;SA1629;SA1642;SA1649;EA0001;EA0009;EA0014;LA0001;LA0003;LA0008;VSTHRD200</NoWarn>
1212
<ImplicitUsings>enable</ImplicitUsings>
1313
</PropertyGroup>
1414

15+
<PropertyGroup>
16+
<Workstream>ServiceDiscovery</Workstream>
17+
<Stage>normal</Stage>
18+
<PackageValidationBaselineVersion>9.5.1</PackageValidationBaselineVersion>
19+
<MinCodeCoverage>75</MinCodeCoverage>
20+
<MinMutationScore>75</MinMutationScore>
21+
</PropertyGroup>
22+
1523
<ItemGroup>
1624
<PackageReference Include="Yarp.ReverseProxy" />
1725
</ItemGroup>

src/Libraries/Microsoft.Extensions.ServiceDiscovery.Yarp/Microsoft.Extensions.ServiceDiscovery.Yarp.json

Whitespace-only changes.

src/Libraries/Microsoft.Extensions.ServiceDiscovery/Microsoft.Extensions.ServiceDiscovery.csproj

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,21 @@
44
<TargetFrameworks>$(TargetFrameworks);netstandard2.0</TargetFrameworks>
55
<IsPackable>true</IsPackable>
66
<Description>Provides extensions to HttpClient that enable service discovery based on configuration.</Description>
7-
<PackageIconFullPath>$(DefaultDotnetIconFullPath)</PackageIconFullPath>
7+
<StrongNameKeyId>Open</StrongNameKeyId>
88
<!-- https://github.com/dotnet/extensions/issues/6871 tracks enabling these -->
99
<NoWarn>$(NoWarn);CS8600;CS8602;CS8604;IDE0040;IDE0055;IDE0058;IDE1006;CA1307;CA1310;CA1849;CA2007;CA2213;SA1204;SA1128;SA1205;SA1405;SA1612;SA1623;SA1625;SA1642;S1144;S1449;S2302;S2692;S3872;S4457;EA0000;EA0009;EA0014;LA0001;LA0003;LA0008;VSTHRD200</NoWarn>
1010
<ImplicitUsings>enable</ImplicitUsings>
1111
<DisableMicrosoftExtensionsLoggingSourceGenerator>false</DisableMicrosoftExtensionsLoggingSourceGenerator>
1212
</PropertyGroup>
1313

14+
<PropertyGroup>
15+
<Workstream>ServiceDiscovery</Workstream>
16+
<Stage>normal</Stage>
17+
<PackageValidationBaselineVersion>9.5.1</PackageValidationBaselineVersion>
18+
<MinCodeCoverage>75</MinCodeCoverage>
19+
<MinMutationScore>75</MinMutationScore>
20+
</PropertyGroup>
21+
1422
<ItemGroup>
1523
<PackageReference Include="Microsoft.Extensions.Http" />
1624
<InternalsVisibleTo Include="Microsoft.Extensions.ServiceDiscovery.Tests" />

src/Libraries/Microsoft.Extensions.ServiceDiscovery/Microsoft.Extensions.ServiceDiscovery.json

Whitespace-only changes.

test/Libraries/Microsoft.Extensions.ServiceDiscovery.Dns.Tests.Fuzzing/Microsoft.Extensions.ServiceDiscovery.Dns.Tests.Fuzzing.csproj

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
<ImplicitUsings>enable</ImplicitUsings>
66
<Nullable>enable</Nullable>
77
<OutputType>Exe</OutputType>
8+
<StrongNameKeyId>Open</StrongNameKeyId>
89
<!-- https://github.com/dotnet/extensions/issues/6871 tracks enabling these -->
910
<NoWarn>$(NoWarn);IDE0040;IDE0061;IDE1006;S5034;SA1400;VSTHRD002</NoWarn>
1011
</PropertyGroup>

0 commit comments

Comments
 (0)