Apply suggestions from code review

Co-authored-by: Chris Ross <[email protected]>

Author: pranavkm

src/Http/Authentication.Core/src/Microsoft.AspNetCore.Authentication.Core.csproj

@@ -4,13 +4,11 @@
     <Description>ASP.NET Core common types used by the various authentication middleware components.</Description>
     <TargetFramework>$(DefaultNetCoreTargetFramework)</TargetFramework>
     <IsAspNetCoreApp>true</IsAspNetCoreApp>
-    <NoWarn>$(NoWarn);CS1591</NoWarn>
+    <NoWarn>$(NoWarn.Replace('1591', ''))</NoWarn>
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <PackageTags>aspnetcore;authentication;security</PackageTags>
     <IsPackable>false</IsPackable>
     <Nullable>enable</Nullable>
-
-    <NoWarn />
   </PropertyGroup>
 
   <ItemGroup>

src/Http/Http.Abstractions/src/ConnectionInfo.cs

@@ -39,15 +39,15 @@ public abstract class ConnectionInfo
         public abstract int LocalPort { get; set; }
 
         /// <summary>
-        /// Gets or sets client certificates associated with the connection.
+        /// Gets or sets the client certificate associated with the connection.
         /// </summary>
         public abstract X509Certificate2? ClientCertificate { get; set; }
 
         /// <summary>
-        /// Asynchronously retries the client certificate.
+        /// Asynchronously retrieves the client certificate.
         /// </summary>
         /// <param name="cancellationToken">The cancellation token to cancel operation.</param>
-        /// <returns>The task object representing the asynchronous operation.</returns>
+        /// <returns>The client certificate if available.</returns>
         public abstract Task<X509Certificate2?> GetClientCertificateAsync(CancellationToken cancellationToken = new CancellationToken());
     }
 }

src/Http/Http.Abstractions/src/Extensions/ResponseTrailerExtensions.cs

@@ -9,7 +9,7 @@
 namespace Microsoft.AspNetCore.Http
 {
     /// <summary>
-    /// Extension methods for <see cref="HttpResponse"/> to work with the 'Trailer' response header.
+    /// Extension methods for <see cref="HttpResponse"/> to work with response trailer headers.
     /// </summary>
     public static class ResponseTrailerExtensions
     {

src/Http/Http.Abstractions/src/Microsoft.AspNetCore.Http.Abstractions.csproj

@@ -14,8 +14,7 @@ Microsoft.AspNetCore.Http.HttpResponse</Description>
     <PackageTags>aspnetcore</PackageTags>
     <IsPackable>false</IsPackable>
     <Nullable>enable</Nullable>
-
-    <NoWarn />
+    <NoWarn>$(NoWarn.Replace('1591', ''))</NoWarn>
   </PropertyGroup>
 
   <ItemGroup>

src/Http/Http.Abstractions/src/PathString.cs

@@ -461,7 +461,7 @@ public static implicit operator PathString(string? s)
             => ConvertFromString(s);
 
         /// <summary>
-        /// Implicitly creates a string frm the given <see cref="PathString"/>.
+        /// Implicitly creates an escaped string from the given <see cref="PathString"/>.
         /// </summary>
         /// <param name="path">A path to implicitly convert.</param>
         public static implicit operator string(PathString path)

src/Http/Metadata/src/Microsoft.AspNetCore.Metadata.csproj

@@ -4,11 +4,10 @@
     <Description>ASP.NET Core metadata.</Description>
     <TargetFrameworks>$(DefaultNetFxTargetFramework);netstandard2.0;$(DefaultNetCoreTargetFramework)</TargetFrameworks>
     <IsAspNetCoreApp>true</IsAspNetCoreApp>
-    <NoWarn>$(NoWarn);CS1591</NoWarn>
+    <NoWarn>$(NoWarn.Replace('1591', ''))</NoWarn>
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
     <PackageTags>aspnetcore</PackageTags>
     <Nullable>enable</Nullable>
-    <NoWarn />
   </PropertyGroup>
 
 </Project>