Browse Source

improve xml docs of some hard to read lists (#44775)

Co-authored-by: Tom Vervoort <[email protected]>
Tom Vervoort 3 years ago
parent
commit
9b154d5081

+ 16 - 8
src/Grpc/JsonTranscoding/src/Microsoft.AspNetCore.Grpc.JsonTranscoding/Internal/JsonTranscodingRouteAdapter.cs

@@ -10,24 +10,32 @@ namespace Microsoft.AspNetCore.Grpc.JsonTranscoding.Internal;
 
 
 /// <summary>
 /// <summary>
 /// Routes on HTTP rule are similar to ASP.NET Core routes but add and remove some features.
 /// Routes on HTTP rule are similar to ASP.NET Core routes but add and remove some features.
-/// - Constraints aren't supported.
-/// - Optional parameters aren't supported.
-/// - Parameters spanning multiple segments are supported.
+/// <list type="bullet">
+/// <item><description>Constraints aren't supported.</description></item>
+/// <item><description>Optional parameters aren't supported.</description></item>
+/// <item><description>Parameters spanning multiple segments are supported.</description></item>
+/// </list>
 ///
 ///
 /// The purpose of this type is to add support for parameters spanning multiple segments and
 /// The purpose of this type is to add support for parameters spanning multiple segments and
 /// anonymous any or catch-all segments. This type transforms an HTTP route into an ASP.NET Core
 /// anonymous any or catch-all segments. This type transforms an HTTP route into an ASP.NET Core
 /// route by rewritting it to a compatible format and providing actions to reconstruct parameters
 /// route by rewritting it to a compatible format and providing actions to reconstruct parameters
 /// that span multiple segments.
 /// that span multiple segments.
 ///
 ///
+/// <para>
 /// For example, consider a multi-segment parameter route:
 /// For example, consider a multi-segment parameter route:
-/// - Before: /v1/{book.name=shelves/*/books/*}
-/// - After: /v1/shelves/{__Complex_book.name_2}/books/{__Complex_book.name_4}
-///
+/// <list type="bullet">
+/// <item><description>Before: /v1/{book.name=shelves/*/books/*}</description></item>
+/// <item><description>After: /v1/shelves/{__Complex_book.name_2}/books/{__Complex_book.name_4}</description></item>
+/// </list>
+/// </para>
+/// 
 /// It is rewritten so that any * or ** segments become ASP.NET Core route parameters. These parameter
 /// It is rewritten so that any * or ** segments become ASP.NET Core route parameters. These parameter
 /// names are never used by the user, and instead they're reconstructed into the final value by the
 /// names are never used by the user, and instead they're reconstructed into the final value by the
 /// adapter and then added to the HttpRequest.RouteValues collection.
 /// adapter and then added to the HttpRequest.RouteValues collection.
-/// - Request URL: /v1/shelves/example-shelf/books/example-book
-/// - Route parameter: book.name = shelves/example-self/books/example-book
+/// <list type="bullet">
+/// <item><description>Request URL: /v1/shelves/example-shelf/books/example-book</description></item>
+/// <item><description>Route parameter: book.name = shelves/example-self/books/example-book</description></item>
+/// </list>
 /// </summary>
 /// </summary>
 internal sealed class JsonTranscodingRouteAdapter
 internal sealed class JsonTranscodingRouteAdapter
 {
 {

+ 22 - 7
src/Http/Http.Features/src/IHttpRequestBodyDetectionFeature.cs

@@ -13,15 +13,30 @@ public interface IHttpRequestBodyDetectionFeature
     /// </summary>
     /// </summary>
     /// <remarks>
     /// <remarks>
     /// This returns true when:
     /// This returns true when:
-    /// - It's an HTTP/1.x request with a non-zero Content-Length or a 'Transfer-Encoding: chunked' header.
-    /// - It's an HTTP/2 request that did not set the END_STREAM flag on the initial headers frame.
+    /// <list type="bullet">
+    /// <item><description>
+    /// It's an HTTP/1.x request with a non-zero Content-Length or a 'Transfer-Encoding: chunked' header.
+    /// </description></item>
+    /// <item><description>
+    /// It's an HTTP/2 request that did not set the END_STREAM flag on the initial headers frame.
+    /// </description></item>
+    /// </list>
     /// The final request body length may still be zero for the chunked or HTTP/2 scenarios.
     /// The final request body length may still be zero for the chunked or HTTP/2 scenarios.
-    ///
+    /// <para>
     /// This returns false when:
     /// This returns false when:
-    /// - It's an HTTP/1.x request with no Content-Length or 'Transfer-Encoding: chunked' header, or the Content-Length is 0.
-    /// - It's an HTTP/1.x request with Connection: Upgrade (e.g. WebSockets). There is no HTTP request body for these requests and
-    ///   no data should be received until after the upgrade.
-    /// - It's an HTTP/2 request that set END_STREAM on the initial headers frame.
+    /// <list type="bullet">
+    /// <item><description>
+    /// It's an HTTP/1.x request with no Content-Length or 'Transfer-Encoding: chunked' header, or the Content-Length is 0.
+    /// </description></item>
+    /// <item><description>
+    /// It's an HTTP/1.x request with Connection: Upgrade (e.g. WebSockets). There is no HTTP request body for these requests and
+    /// no data should be received until after the upgrade.
+    /// </description></item>
+    /// <item><description>
+    /// It's an HTTP/2 request that set END_STREAM on the initial headers frame.
+    /// </description></item>
+    /// </list>
+    /// </para>
     /// When false, the request body should never return data.
     /// When false, the request body should never return data.
     /// </remarks>
     /// </remarks>
     bool CanHaveBody { get; }
     bool CanHaveBody { get; }