diff --git a/NuGetPackageVerifier.json b/NuGetPackageVerifier.json
index d77ace68be..21c36d6f60 100644
--- a/NuGetPackageVerifier.json
+++ b/NuGetPackageVerifier.json
@@ -1,12 +1,7 @@
 {
     "adx": { // Packages written by the ADX team and that ship on NuGet.org
         "rules": [
-            "AssemblyHasDocumentFileRule",
-            "AssemblyHasVersionAttributesRule",
-            "AssemblyHasServicingAttributeRule",
-            "AssemblyHasNeutralResourcesLanguageAttributeRule",
-            "SatellitePackageRule",
-            "StrictSemanticVersionValidationRule"
+            "AdxVerificationCompositeRule"
         ],
         "packages": {
             "Microsoft.AspNetCore.Mvc": { },
@@ -27,12 +22,7 @@
     },
     "Default": { // Rules to run for packages not listed in any other set.
         "rules": [
-            "AssemblyHasDocumentFileRule",
-            "AssemblyHasVersionAttributesRule",
-            "AssemblyHasServicingAttributeRule",
-            "AssemblyHasNeutralResourcesLanguageAttributeRule",
-            "SatellitePackageRule",
-            "StrictSemanticVersionValidationRule"
+            "DefaultCompositeRule"
         ]
     }
 }
diff --git a/src/Microsoft.AspNetCore.Mvc.Abstractions/ActionConstraints/ActionConstraintProviderContext.cs b/src/Microsoft.AspNetCore.Mvc.Abstractions/ActionConstraints/ActionConstraintProviderContext.cs
index 01ac6d223c..5618cfe2b5 100644
--- a/src/Microsoft.AspNetCore.Mvc.Abstractions/ActionConstraints/ActionConstraintProviderContext.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Abstractions/ActionConstraints/ActionConstraintProviderContext.cs
@@ -16,6 +16,7 @@ namespace Microsoft.AspNetCore.Mvc.ActionConstraints
         /// <summary>
         /// Creates a new <see cref="ActionConstraintProviderContext"/>.
         /// </summary>
+        /// <param name="context">The <see cref="Http.HttpContext"/> associated with the request.</param>
         /// <param name="action">The <see cref="ActionDescriptor"/> for which constraints are being created.</param>
         /// <param name="items">The list of <see cref="ActionConstraintItem"/> objects.</param>
         public ActionConstraintProviderContext(
@@ -43,6 +44,9 @@ namespace Microsoft.AspNetCore.Mvc.ActionConstraints
             Results = items;
         }
 
+        /// <summary>
+        /// The <see cref="Http.HttpContext"/> associated with the request.
+        /// </summary>
         public HttpContext HttpContext { get; }
 
         /// <summary>
diff --git a/src/Microsoft.AspNetCore.Mvc.Abstractions/Filters/IOrderedFilter.cs b/src/Microsoft.AspNetCore.Mvc.Abstractions/Filters/IOrderedFilter.cs
index b29788592d..452cb70d63 100644
--- a/src/Microsoft.AspNetCore.Mvc.Abstractions/Filters/IOrderedFilter.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Abstractions/Filters/IOrderedFilter.cs
@@ -21,7 +21,7 @@ namespace Microsoft.AspNetCore.Mvc.Filters
         /// </para>
         /// <para>
         /// Synchronous filters, such as <see cref="IActionFilter"/>, have a before-method, such as
-        /// <see cref="IActionFilter.OnActionExecuting)"/>, and an after-method, such as
+        /// <see cref="IActionFilter.OnActionExecuting"/>, and an after-method, such as
         /// <see cref="IActionFilter.OnActionExecuted"/>. A synchronous filter with a lower numeric <see cref="Order"/>
         /// value will have its before-method executed before that of a filter with a higher value of 
         /// <see cref="Order"/>. During the after-stage of the filter, a synchronous filter with a lower
diff --git a/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelBindingContext.cs b/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelBindingContext.cs
index 35690dc1e1..46bba22316 100644
--- a/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelBindingContext.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelBindingContext.cs
@@ -139,12 +139,12 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         public abstract NestedScope EnterNestedScope();
 
         /// <summary>
-        /// Removes a layer of state pushed by calling <see cref="EnterNestedScope"/>.
+        /// Removes a layer of state pushed by calling <see cref="M:EnterNestedScope"/>.
         /// </summary>
         protected abstract void ExitNestedScope();
 
         /// <summary>
-        /// Return value of <see cref="ModelBindingContext.EnterNestedScope"/>. Should be disposed
+        /// Return value of <see cref="M:EnterNestedScope"/>. Should be disposed
         /// by caller when child binding context state should be popped off of 
         /// the <see cref="ModelBindingContext"/>.
         /// </summary>
@@ -162,7 +162,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
             }
 
             /// <summary>
-            /// Exits the <see cref="NestedScope"/> created by calling <see cref="ModelBindingContext.EnterNestedScope"/>.
+            /// Exits the <see cref="NestedScope"/> created by calling <see cref="M:EnterNestedScope"/>.
             /// </summary>
             public void Dispose()
             {
diff --git a/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelMetadata.cs b/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelMetadata.cs
index 923bcd158c..c21f6489c0 100644
--- a/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelMetadata.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelMetadata.cs
@@ -143,7 +143,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         /// <see cref="UnderlyingOrModelType"/>.
         /// </summary>
         /// <value>
-        /// An <see cref="IEnumerable{KeyValuePair{EnumGroupAndName, string}"/> of mappings between
+        /// An <see cref="IEnumerable{T}"/> of <see cref="KeyValuePair{EnumGroupAndName, String}"/> of mappings between
         /// <see cref="Enum"/> field groups, names and values. <c>null</c> if <see cref="IsEnum"/> is <c>false</c>.
         /// </value>
         public abstract IEnumerable<KeyValuePair<EnumGroupAndName, string>> EnumGroupedDisplayNamesAndValues { get; }
@@ -152,7 +152,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         /// Gets the names and values of all <see cref="Enum"/> values in <see cref="UnderlyingOrModelType"/>.
         /// </summary>
         /// <value>
-        /// An <see cref="IReadOnlyDictionary{string, string}"/> of mappings between <see cref="Enum"/> field names
+        /// An <see cref="IReadOnlyDictionary{String, String}"/> of mappings between <see cref="Enum"/> field names
         /// and values. <c>null</c> if <see cref="IsEnum"/> is <c>false</c>.
         /// </value>
         public abstract IReadOnlyDictionary<string, string> EnumNamesAndValues { get; }
diff --git a/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelStateDictionary.cs b/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelStateDictionary.cs
index 4d97e7d8eb..f6bdaf83dd 100644
--- a/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelStateDictionary.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelStateDictionary.cs
@@ -110,7 +110,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
 
         /// <summary>
         /// Gets the number of errors added to this instance of <see cref="ModelStateDictionary"/> via
-        /// <see cref="AddModelError"/> or <see cref="TryAddModelError"/>.
+        /// <see cref="M:AddModelError"/> or <see cref="M:TryAddModelError"/>.
         /// </summary>
         public int ErrorCount { get; private set; }
 
@@ -203,6 +203,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         /// </summary>
         /// <param name="key">The key of the <see cref="ModelStateEntry"/> to add errors to.</param>
         /// <param name="exception">The <see cref="Exception"/> to add.</param>
+        /// <param name="metadata">The <see cref="ModelMetadata"/> associated with the model.</param>
         public void AddModelError(string key, Exception exception, ModelMetadata metadata)
         {
             if (key == null)
@@ -230,6 +231,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         /// </summary>
         /// <param name="key">The key of the <see cref="ModelStateEntry"/> to add errors to.</param>
         /// <param name="exception">The <see cref="Exception"/> to add.</param>
+        /// <param name="metadata">The <see cref="ModelMetadata"/> associated with the model.</param>
         /// <returns>
         /// <c>True</c> if the given error was added, <c>false</c> if the error was ignored.
         /// See <see cref="MaxAllowedErrors"/>.
@@ -449,9 +451,9 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         /// the <see cref="ModelStateEntry"/> with the specified <paramref name="key"/>.
         /// </summary>
         /// <param name="key">The key for the <see cref="ModelStateEntry"/> entry.</param>
-        /// <param name="rawvalue">The raw value for the <see cref="ModelStateEntry"/> entry.</param>
+        /// <param name="rawValue">The raw value for the <see cref="ModelStateEntry"/> entry.</param>
         /// <param name="attemptedValue">
-        /// The values of <param name="rawValue"/> in a comma-separated <see cref="string"/>.
+        /// The values of <paramref name="rawValue"/> in a comma-separated <see cref="string"/>.
         /// </param>
         public void SetModelValue(string key, object rawValue, string attemptedValue)
         {
diff --git a/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/Validation/ClientValidatorProviderContext.cs b/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/Validation/ClientValidatorProviderContext.cs
index fcfcadf14d..68ced77b45 100644
--- a/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/Validation/ClientValidatorProviderContext.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/Validation/ClientValidatorProviderContext.cs
@@ -44,7 +44,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding.Validation
         /// <summary>
         /// Gets the list of <see cref="ClientValidatorItem"/> instances. <see cref="IClientModelValidatorProvider"/>
         /// instances should add the appropriate <see cref="ClientValidatorItem.Validator"/> properties when
-        /// <see cref="IClientModelValidatorProvider.CreateValidators(ClientValidatorProviderContext)()"/>
+        /// <see cref="IClientModelValidatorProvider.CreateValidators(ClientValidatorProviderContext)"/>
         /// is called.
         /// </summary>
         public IList<ClientValidatorItem> Results { get; }
diff --git a/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ValueProviderResult.cs b/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ValueProviderResult.cs
index d0e3d206b7..69b1b27af6 100644
--- a/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ValueProviderResult.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ValueProviderResult.cs
@@ -21,7 +21,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
     /// multiple values were submitted.
     /// </para>
     /// <para>
-    /// Treat <see cref="ValueProviderResult"/> as an <see cref="IEnumerable{string}"/> to consume all values,
+    /// Treat <see cref="ValueProviderResult"/> as an <see cref="IEnumerable{String}"/> to consume all values,
     /// regardless of whether a single value or multiple values were submitted.
     /// </para>
     /// </remarks>
@@ -46,7 +46,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         /// <summary>
         /// Creates a new <see cref="ValueProviderResult"/>.
         /// </summary>
-        /// <param name="value">The submitted value.</param>
+        /// <param name="values">The submitted values.</param>
         /// <param name="culture">The <see cref="CultureInfo"/> associated with this value.</param>
         public ValueProviderResult(StringValues values, CultureInfo culture)
         {
@@ -134,9 +134,9 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         }
 
         /// <summary>
-        /// Gets an <see cref="IEnumerator{string}"/> for this <see cref="ValueProviderResult"/>.
+        /// Gets an <see cref="IEnumerator{String}"/> for this <see cref="ValueProviderResult"/>.
         /// </summary>
-        /// <returns>An <see cref="IEnumerator{string}"/>.</returns>
+        /// <returns>An <see cref="IEnumerator{String}"/>.</returns>
         public IEnumerator<string> GetEnumerator()
         {
             return ((IEnumerable<string>)Values).GetEnumerator();
diff --git a/src/Microsoft.AspNetCore.Mvc.Abstractions/project.json b/src/Microsoft.AspNetCore.Mvc.Abstractions/project.json
index e46d5043cb..439c1b112a 100644
--- a/src/Microsoft.AspNetCore.Mvc.Abstractions/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.Abstractions/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Routing.Abstractions": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.ApiExplorer/ApiDescriptionGroupCollectionProvider.cs b/src/Microsoft.AspNetCore.Mvc.ApiExplorer/ApiDescriptionGroupCollectionProvider.cs
index 4a1ae7ae98..1d7976d5bd 100644
--- a/src/Microsoft.AspNetCore.Mvc.ApiExplorer/ApiDescriptionGroupCollectionProvider.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ApiExplorer/ApiDescriptionGroupCollectionProvider.cs
@@ -22,7 +22,7 @@ namespace Microsoft.AspNetCore.Mvc.ApiExplorer
         /// The <see cref="IActionDescriptorCollectionProvider"/>.
         /// </param>
         /// <param name="apiDescriptionProviders">
-        /// The <see cref="IEnumerable{IApiDescriptionProvider}}"/>.
+        /// The <see cref="IEnumerable{IApiDescriptionProvider}"/>.
         /// </param>
         public ApiDescriptionGroupCollectionProvider(
             IActionDescriptorCollectionProvider actionDescriptorCollectionProvider,
diff --git a/src/Microsoft.AspNetCore.Mvc.ApiExplorer/DefaultApiDescriptionProvider.cs b/src/Microsoft.AspNetCore.Mvc.ApiExplorer/DefaultApiDescriptionProvider.cs
index 11740057a9..8215cfe67a 100644
--- a/src/Microsoft.AspNetCore.Mvc.ApiExplorer/DefaultApiDescriptionProvider.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ApiExplorer/DefaultApiDescriptionProvider.cs
@@ -589,13 +589,6 @@ namespace Microsoft.AspNetCore.Mvc.ApiExplorer
                 Visit(context, source, containerName: string.Empty);
             }
 
-            /// <summary>
-            /// Visits a node in a model, and attempts to create <see cref="ApiParameterDescription"/> for any
-            /// model properties that are leaf nodes (see comments).
-            /// </summary>
-            /// <param name="modelMetadata">The metadata for the model.</param>
-            /// <param name="ambientSource">The <see cref="BindingSource"/> from the ambient context.</param>
-            /// <param name="containerName">The current name prefix (to prepend to property names).</param>
             private void Visit(
                 ApiParameterDescriptionContext bindingContext,
                 BindingSource ambientSource,
diff --git a/src/Microsoft.AspNetCore.Mvc.ApiExplorer/project.json b/src/Microsoft.AspNetCore.Mvc.ApiExplorer/project.json
index 0f8cb2957b..fa0664a164 100644
--- a/src/Microsoft.AspNetCore.Mvc.ApiExplorer/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.ApiExplorer/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Mvc.Core": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/ChallengeResult.cs b/src/Microsoft.AspNetCore.Mvc.Core/ChallengeResult.cs
index 2d8b6c8277..5f1fb28767 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/ChallengeResult.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/ChallengeResult.cs
@@ -12,7 +12,7 @@ using Microsoft.Extensions.Logging;
 namespace Microsoft.AspNetCore.Mvc
 {
     /// <summary>
-    /// An <see cref="ActionResult"/> that on execution invokes <see cref="AuthenticationManager.ChallengeAsync"/>.
+    /// An <see cref="ActionResult"/> that on execution invokes <see cref="M:AuthenticationManager.ChallengeAsync"/>.
     /// </summary>
     public class ChallengeResult : ActionResult
     {
@@ -38,7 +38,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Initializes a new instance of <see cref="ChallengeResult"/> with the
         /// specified authentication schemes.
         /// </summary>
-        /// <param name="authenticationScheme">The authentication schemes to challenge.</param>
+        /// <param name="authenticationSchemes">The authentication schemes to challenge.</param>
         public ChallengeResult(IList<string> authenticationSchemes)
             : this(authenticationSchemes, properties: null)
         {
@@ -71,7 +71,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Initializes a new instance of <see cref="ChallengeResult"/> with the
         /// specified authentication schemes and <paramref name="properties"/>.
         /// </summary>
-        /// <param name="authenticationScheme">The authentication scheme to challenge.</param>
+        /// <param name="authenticationSchemes">The authentication scheme to challenge.</param>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the authentication
         /// challenge.</param>
         public ChallengeResult(IList<string> authenticationSchemes, AuthenticationProperties properties)
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/ControllerBase.cs b/src/Microsoft.AspNetCore.Mvc.Core/ControllerBase.cs
index e44e9795e3..e60e2d2fa6 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/ControllerBase.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/ControllerBase.cs
@@ -935,7 +935,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <summary>
         /// Creates a <see cref="ChallengeResult"/> with the specified authentication schemes.
         /// </summary>
-        /// <param name="authenticationScheme">The authentication schemes to challenge.</param>
+        /// <param name="authenticationSchemes">The authentication schemes to challenge.</param>
         /// <returns>The created <see cref="ChallengeResult"/> for the response.</returns>
         [NonAction]
         public virtual ChallengeResult Challenge(IList<string> authenticationSchemes)
@@ -967,7 +967,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Creates a <see cref="ChallengeResult"/> with the specified specified authentication schemes and
         /// <paramref name="properties" />.
         /// </summary>
-        /// <param name="authenticationScheme">The authentication schemes to challenge.</param>
+        /// <param name="authenticationSchemes">The authentication schemes to challenge.</param>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the authentication
         /// challenge.</param>
         /// <returns>The created <see cref="ChallengeResult"/> for the response.</returns>
@@ -997,7 +997,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <summary>
         /// Creates a <see cref="ForbidResult"/> with the specified authentication schemes.
         /// </summary>
-        /// <param name="authenticationScheme">The authentication schemes to challenge.</param>
+        /// <param name="authenticationSchemes">The authentication schemes to challenge.</param>
         /// <returns>The created <see cref="ForbidResult"/> for the response.</returns>
         [NonAction]
         public virtual ForbidResult Forbid(IList<string> authenticationSchemes)
@@ -1029,7 +1029,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Creates a <see cref="ForbidResult"/> with the specified specified authentication schemes and
         /// <paramref name="properties" />.
         /// </summary>
-        /// <param name="authenticationScheme">The authentication schemes to challenge.</param>
+        /// <param name="authenticationSchemes">The authentication schemes to challenge.</param>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the authentication
         /// challenge.</param>
         /// <returns>The created <see cref="ForbidResult"/> for the response.</returns>
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/Controllers/IControllerActivator.cs b/src/Microsoft.AspNetCore.Mvc.Core/Controllers/IControllerActivator.cs
index 46c7259638..72ff618b48 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/Controllers/IControllerActivator.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/Controllers/IControllerActivator.cs
@@ -19,6 +19,7 @@ namespace Microsoft.AspNetCore.Mvc.Controllers
         /// <summary>
         /// Releases a controller.
         /// </summary>
+        /// <param name="context">The <see cref="ControllerContext"/> for the executing action.</param>
         /// <param name="controller">The controller to release.</param>
         void Release(ControllerContext context, object controller);
     }
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/Controllers/IControllerFactory.cs b/src/Microsoft.AspNetCore.Mvc.Core/Controllers/IControllerFactory.cs
index e4a16086f1..ae66e446a7 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/Controllers/IControllerFactory.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/Controllers/IControllerFactory.cs
@@ -18,6 +18,7 @@ namespace Microsoft.AspNetCore.Mvc.Controllers
         /// <summary>
         /// Releases a controller instance.
         /// </summary>
+        /// <param name="context"><see cref="ControllerContext"/> for the executing action.</param>
         /// <param name="controller">The controller.</param>
         void ReleaseController(ControllerContext context, object controller);
     }
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/ForbidResult.cs b/src/Microsoft.AspNetCore.Mvc.Core/ForbidResult.cs
index 37b86e3e42..6effcb8673 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/ForbidResult.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/ForbidResult.cs
@@ -12,7 +12,7 @@ using Microsoft.Extensions.Logging;
 namespace Microsoft.AspNetCore.Mvc
 {
     /// <summary>
-    /// An <see cref="ActionResult"/> that on execution invokes <see cref="AuthenticationManager.ForbidAsync"/>.
+    /// An <see cref="ActionResult"/> that on execution invokes <see cref="M:AuthenticationManager.ForbidAsync"/>.
     /// </summary>
     public class ForbidResult : ActionResult
     {
@@ -38,7 +38,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Initializes a new instance of <see cref="ForbidResult"/> with the
         /// specified authentication schemes.
         /// </summary>
-        /// <param name="authenticationScheme">The authentication schemes to challenge.</param>
+        /// <param name="authenticationSchemes">The authentication schemes to challenge.</param>
         public ForbidResult(IList<string> authenticationSchemes)
             : this(authenticationSchemes, properties: null)
         {
@@ -71,7 +71,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Initializes a new instance of <see cref="ForbidResult"/> with the
         /// specified authentication schemes and <paramref name="properties"/>.
         /// </summary>
-        /// <param name="authenticationScheme">The authentication scheme to challenge.</param>
+        /// <param name="authenticationSchemes">The authentication scheme to challenge.</param>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the authentication
         /// challenge.</param>
         public ForbidResult(IList<string> authenticationSchemes, AuthenticationProperties properties)
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/Formatters/TextOutputFormatter.cs b/src/Microsoft.AspNetCore.Mvc.Core/Formatters/TextOutputFormatter.cs
index 808c906e13..05af51ffcb 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/Formatters/TextOutputFormatter.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/Formatters/TextOutputFormatter.cs
@@ -58,7 +58,7 @@ namespace Microsoft.AspNetCore.Mvc.Formatters
 
         /// <summary>
         /// Determines the best <see cref="Encoding"/> amongst the supported encodings
-        /// for reading or writing an HTTP entity body based on the provided <paramref name="contentTypeHeader"/>.
+        /// for reading or writing an HTTP entity body based on the provided content type.
         /// </summary>
         /// <param name="context">The formatter context associated with the call.
         /// </param>
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/Internal/ActionSelector.cs b/src/Microsoft.AspNetCore.Mvc.Core/Internal/ActionSelector.cs
index c6a6685399..5c8d4e84e3 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/Internal/ActionSelector.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/Internal/ActionSelector.cs
@@ -26,7 +26,8 @@ namespace Microsoft.AspNetCore.Mvc.Internal
         /// Creates a new <see cref="ActionSelector"/>.
         /// </summary>
         /// <param name="decisionTreeProvider">The <see cref="IActionSelectorDecisionTreeProvider"/>.</param>
-        /// <param name="actionConstraintProviders">The set of <see cref="IActionInvokerProvider"/> instances.</param>
+        /// <param name="actionConstraintCache">The <see cref="ActionConstraintCache"/> that
+        /// providers a set of <see cref="IActionConstraint"/> instances.</param>
         /// <param name="loggerFactory">The <see cref="ILoggerFactory"/>.</param>
         public ActionSelector(
             IActionSelectorDecisionTreeProvider decisionTreeProvider,
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/Internal/DefaultObjectValidator.cs b/src/Microsoft.AspNetCore.Mvc.Core/Internal/DefaultObjectValidator.cs
index 5c86e2a8eb..dbdceb5f85 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/Internal/DefaultObjectValidator.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/Internal/DefaultObjectValidator.cs
@@ -19,6 +19,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
         /// Initializes a new instance of <see cref="DefaultObjectValidator"/>.
         /// </summary>
         /// <param name="modelMetadataProvider">The <see cref="IModelMetadataProvider"/>.</param>
+        /// <param name="validatorCache">The <see cref="ValidatorCache"/>.</param>
         public DefaultObjectValidator(
             IModelMetadataProvider modelMetadataProvider,
             ValidatorCache validatorCache)
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/Internal/ITypeActivatorCache.cs b/src/Microsoft.AspNetCore.Mvc.Core/Internal/ITypeActivatorCache.cs
index b77c47e76c..a097b29910 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/Internal/ITypeActivatorCache.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/Internal/ITypeActivatorCache.cs
@@ -15,8 +15,8 @@ namespace Microsoft.AspNetCore.Mvc.Internal
         /// Creates an instance of <typeparamref name="TInstance"/>.
         /// </summary>
         /// <param name="serviceProvider">The <see cref="IServiceProvider"/> used to resolve dependencies for
-        /// <paramref name="implementationType"/>.</param>
-        /// <param name="implementationType">The <see cref="Type"/> of the <typeparamref name="TInstance"/> to create.</param>
+        /// <paramref name="optionType"/>.</param>
+        /// <param name="optionType">The <see cref="Type"/> of the <typeparamref name="TInstance"/> to create.</param>
         TInstance CreateInstance<TInstance>(IServiceProvider serviceProvider, Type optionType);
     }
 }
\ No newline at end of file
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/Internal/MemoryPoolHttpRequestStreamReaderFactory.cs b/src/Microsoft.AspNetCore.Mvc.Core/Internal/MemoryPoolHttpRequestStreamReaderFactory.cs
index d006ebcb54..108956b40b 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/Internal/MemoryPoolHttpRequestStreamReaderFactory.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/Internal/MemoryPoolHttpRequestStreamReaderFactory.cs
@@ -26,10 +26,10 @@ namespace Microsoft.AspNetCore.Mvc.Internal
         /// Creates a new <see cref="MemoryPoolHttpRequestStreamReaderFactory"/>.
         /// </summary>
         /// <param name="bytePool">
-        /// The <see cref="ArrayPool{byte}"/> for creating <see cref="byte"/> buffers.
+        /// The <see cref="ArrayPool{Byte}"/> for creating <see cref="T:byte[]"/> buffers.
         /// </param>
         /// <param name="charPool">
-        /// The <see cref="ArrayPool{char}"/> for creating <see cref="char"/> buffers.
+        /// The <see cref="ArrayPool{Char}"/> for creating <see cref="T:char[]"/> buffers.
         /// </param>
         public MemoryPoolHttpRequestStreamReaderFactory(
             ArrayPool<byte> bytePool,
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/Internal/MemoryPoolHttpResponseStreamWriterFactory.cs b/src/Microsoft.AspNetCore.Mvc.Core/Internal/MemoryPoolHttpResponseStreamWriterFactory.cs
index 4d66f3af3a..95e0615613 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/Internal/MemoryPoolHttpResponseStreamWriterFactory.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/Internal/MemoryPoolHttpResponseStreamWriterFactory.cs
@@ -26,10 +26,10 @@ namespace Microsoft.AspNetCore.Mvc.Internal
         /// Creates a new <see cref="MemoryPoolHttpResponseStreamWriterFactory"/>.
         /// </summary>
         /// <param name="bytePool">
-        /// The <see cref="ArrayPool{byte}"/> for creating <see cref="byte"/> buffers.
+        /// The <see cref="ArrayPool{Byte}"/> for creating <see cref="byte"/> buffers.
         /// </param>
         /// <param name="charPool">
-        /// The <see cref="ArrayPool{char}"/> for creating <see cref="char"/> buffers.
+        /// The <see cref="ArrayPool{Char}"/> for creating <see cref="char"/> buffers.
         /// </param>
         public MemoryPoolHttpResponseStreamWriterFactory(
             ArrayPool<byte> bytePool,
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/LocalRedirectResult.cs b/src/Microsoft.AspNetCore.Mvc.Core/LocalRedirectResult.cs
index dbc32011c0..b42b259df7 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/LocalRedirectResult.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/LocalRedirectResult.cs
@@ -31,7 +31,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Initializes a new instance of the <see cref="LocalRedirectResult"/> class with the values
         /// provided.
         /// </summary>
-        /// <param name="url">The local URL to redirect to.</param>
+        /// <param name="localUrl">The local URL to redirect to.</param>
         /// <param name="permanent">Specifies whether the redirect should be permanent (301) or temporary (302).</param>
         public LocalRedirectResult(string localUrl, bool permanent)
         {
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/ModelBindingHelper.cs b/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/ModelBindingHelper.cs
index 5b120d9f05..6c699515a6 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/ModelBindingHelper.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/ModelBindingHelper.cs
@@ -632,12 +632,13 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         /// <summary>
         /// Clears <see cref="ModelStateDictionary"/> entries for <see cref="ModelMetadata"/>.
         /// </summary>
-        /// <param name="modelMetadata">The <see cref="ModelMetadata"/>.</param>
+        /// <param name="modelType">The <see cref="Type"/> of the model.</param>
+        /// <param name="modelState">The <see cref="ModelStateDictionary"/> associated with the model.</param>
+        /// <param name="metadataProvider">The <see cref="IModelMetadataProvider"/>.</param>
         /// <param name="modelKey">The entry to clear. </param>
-        /// <param name="modelMetadataProvider">The <see cref="IModelMetadataProvider"/>.</param>
         public static void ClearValidationStateForModel(
             Type modelType,
-            ModelStateDictionary modelstate,
+            ModelStateDictionary modelState,
             IModelMetadataProvider metadataProvider,
             string modelKey)
         {
@@ -646,9 +647,9 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
                 throw new ArgumentNullException(nameof(modelType));
             }
 
-            if (modelstate == null)
+            if (modelState == null)
             {
-                throw new ArgumentNullException(nameof(modelstate));
+                throw new ArgumentNullException(nameof(modelState));
             }
 
             if (metadataProvider == null)
@@ -656,18 +657,18 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
                 throw new ArgumentNullException(nameof(metadataProvider));
             }
 
-            ClearValidationStateForModel(metadataProvider.GetMetadataForType(modelType), modelstate, modelKey);
+            ClearValidationStateForModel(metadataProvider.GetMetadataForType(modelType), modelState, modelKey);
         }
 
         /// <summary>
         /// Clears <see cref="ModelStateDictionary"/> entries for <see cref="ModelMetadata"/>.
         /// </summary>
         /// <param name="modelMetadata">The <see cref="ModelMetadata"/>.</param>
+        /// <param name="modelState">The <see cref="ModelStateDictionary"/> associated with the model.</param>
         /// <param name="modelKey">The entry to clear. </param>
-        /// <param name="modelMetadataProvider">The <see cref="IModelMetadataProvider"/>.</param>
         public static void ClearValidationStateForModel(
             ModelMetadata modelMetadata,
-            ModelStateDictionary modelstate,
+            ModelStateDictionary modelState,
             string modelKey)
         {
             if (modelMetadata == null)
@@ -675,9 +676,9 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
                 throw new ArgumentNullException(nameof(modelMetadata));
             }
 
-            if (modelstate == null)
+            if (modelState == null)
             {
-                throw new ArgumentNullException(nameof(modelstate));
+                throw new ArgumentNullException(nameof(modelState));
             }
 
             if (string.IsNullOrEmpty(modelKey))
@@ -692,7 +693,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
                     //
                     // In the unlikely case that multiple top-level collections where bound to the empty prefix,
                     // you're just out of luck.
-                    foreach (var kvp in modelstate)
+                    foreach (var kvp in modelState)
                     {
                         if (kvp.Key.Length > 0 && kvp.Key[0] == '[')
                         {
@@ -706,14 +707,14 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
                 {
                     foreach (var property in modelMetadata.Properties)
                     {
-                        modelstate.ClearValidationState(property.BinderModelName ?? property.PropertyName);
+                        modelState.ClearValidationState(property.BinderModelName ?? property.PropertyName);
                     }
                 }
                 else
                 {
                     // Simple types bind to a single entry. So clear the entry with the empty-key, in the
                     // unlikely event that it has errors.
-                    var entry = modelstate[string.Empty];
+                    var entry = modelState[string.Empty];
                     if (entry != null)
                     {
                         entry.Errors.Clear();
@@ -726,7 +727,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
                 // If model key is non-empty, we just want to clear all keys with that prefix. We expect
                 // model binding to have only used this key (and suffixes) for all entries related to
                 // this model.
-                modelstate.ClearValidationState(modelKey);
+                modelState.ClearValidationState(modelKey);
             }
         }
 
@@ -849,10 +850,10 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         }
 
         /// <summary>
-        /// Converts the provided <paramref name="value"/> to a value of <see cref="Type"/> <param name="type"/>
+        /// Converts the provided <paramref name="value"/> to a value of <see cref="Type"/> <paramref name="type"/>
         /// using the <see cref="CultureInfo.InvariantCulture"/>.
         /// </summary>
-        /// <param name="value">The value to convert."/></param>
+        /// <param name="value">The value to convert.</param>
         /// <param name="type">The <see cref="Type"/> for conversion.</param>
         /// <returns>
         /// The converted value or <c>null</c> if the value could not be converted.
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Validation/IObjectModelValidator.cs b/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Validation/IObjectModelValidator.cs
index 2ed2d9f8f8..d0ffa99264 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Validation/IObjectModelValidator.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Validation/IObjectModelValidator.cs
@@ -15,7 +15,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding.Validation
         /// <param name="validatorProvider">The <see cref="IModelValidatorProvider"/>.</param>
         /// <param name="validationState">The <see cref="ValidationStateDictionary"/>. May be null.</param>
         /// <param name="prefix">
-        /// The model prefix. Used to map the model object to entries in <paramref name="modelState"/>.
+        /// The model prefix. Used to map the model object to entries in <paramref name="validationState"/>.
         /// </param>
         /// <param name="model">The model object.</param>
         void Validate(
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Validation/ValidationVisitor.cs b/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Validation/ValidationVisitor.cs
index 3e66d91022..4e6ba85f1d 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Validation/ValidationVisitor.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Validation/ValidationVisitor.cs
@@ -34,6 +34,8 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding.Validation
         /// </summary>
         /// <param name="actionContext">The <see cref="ActionContext"/> associated with the current request.</param>
         /// <param name="validatorProvider">The <see cref="IModelValidatorProvider"/>.</param>
+        /// <param name="validatorCache">The <see cref="ValidatorCache"/> that provides a list of <see cref="IModelValidator"/>s.</param>
+        /// <param name="metadataProvider">The provider used for reading metadata for the model type.</param>
         /// <param name="validationState">The <see cref="ValidationStateDictionary"/>.</param>
         public ValidationVisitor(
             ActionContext actionContext,
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/UrlHelperExtensions.cs b/src/Microsoft.AspNetCore.Mvc.Core/UrlHelperExtensions.cs
index 5171f7ec4f..ff7bab72c4 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/UrlHelperExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Core/UrlHelperExtensions.cs
@@ -31,6 +31,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <summary>
         /// Generates a fully qualified or absolute URL for an action method by using the specified action name.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="action">The name of the action method.</param>
         /// <returns>The fully qualified or absolute URL to an action method.</returns>
         public static string Action(this IUrlHelper helper, string action)
@@ -47,6 +48,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Generates a fully qualified or absolute URL for an action method by using the specified action name,
         /// and route values.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="action">The name of the action method.</param>
         /// <param name="values">An object that contains route values.</param>
         /// <returns>The fully qualified or absolute URL to an action method.</returns>
@@ -64,6 +66,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Generates a fully qualified or absolute URL for an action method by using the specified action name,
         /// and controller name.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="action">The name of the action method.</param>
         /// <param name="controller">The name of the controller.</param>
         /// <returns>The fully qualified or absolute URL to an action method.</returns>
@@ -81,6 +84,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Generates a fully qualified or absolute URL for an action method by using the specified action name,
         /// controller name, and route values.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="action">The name of the action method.</param>
         /// <param name="controller">The name of the controller.</param>
         /// <param name="values">An object that contains route values.</param>
@@ -99,6 +103,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Generates a fully qualified or absolute URL for an action method by using the specified action name,
         /// controller name, route values, and protocol to use.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="action">The name of the action method.</param>
         /// <param name="controller">The name of the controller.</param>
         /// <param name="values">An object that contains route values.</param>
@@ -123,6 +128,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Generates a fully qualified or absolute URL for an action method by using the specified action name,
         /// controller name, route values, protocol to use, and host name.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="action">The name of the action method.</param>
         /// <param name="controller">The name of the controller.</param>
         /// <param name="values">An object that contains route values.</param>
@@ -149,6 +155,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Generates a fully qualified or absolute URL for an action method by using the specified action name,
         /// controller name, route values, protocol to use, host name and fragment.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="action">The name of the action method.</param>
         /// <param name="controller">The name of the controller.</param>
         /// <param name="values">An object that contains route values.</param>
@@ -184,6 +191,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <summary>
         /// Generates a fully qualified or absolute URL for the specified route values.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="values">An object that contains route values.</param>
         /// <returns>The fully qualified or absolute URL.</returns>
         public static string RouteUrl(this IUrlHelper helper, object values)
@@ -199,6 +207,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <summary>
         /// Generates a fully qualified or absolute URL for the specified route name.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="routeName">The name of the route that is used to generate URL.</param>
         /// <returns>The fully qualified or absolute URL.</returns>
         public static string RouteUrl(this IUrlHelper helper, string routeName)
@@ -215,6 +224,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Generates a fully qualified or absolute URL for the specified route values by
         /// using the specified route name.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="routeName">The name of the route that is used to generate URL.</param>
         /// <param name="values">An object that contains route values.</param>
         /// <returns>The fully qualified or absolute URL.</returns>
@@ -232,6 +242,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Generates a fully qualified or absolute URL for the specified route values by
         /// using the specified route name, and protocol to use.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="routeName">The name of the route that is used to generate URL.</param>
         /// <param name="values">An object that contains route values.</param>
         /// <param name="protocol">The protocol for the URL, such as "http" or "https".</param>
@@ -254,6 +265,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Generates a fully qualified or absolute URL for the specified route values by
         /// using the specified route name, protocol to use, and host name.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="routeName">The name of the route that is used to generate URL.</param>
         /// <param name="values">An object that contains route values.</param>
         /// <param name="protocol">The protocol for the URL, such as "http" or "https".</param>
@@ -278,6 +290,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// Generates a fully qualified or absolute URL for the specified route values by
         /// using the specified route name, protocol to use, host name and fragment.
         /// </summary>
+        /// <param name="helper">The <see cref="IUrlHelper"/>.</param>
         /// <param name="routeName">The name of the route that is used to generate URL.</param>
         /// <param name="values">An object that contains route values.</param>
         /// <param name="protocol">The protocol for the URL, such as "http" or "https".</param>
diff --git a/src/Microsoft.AspNetCore.Mvc.Core/project.json b/src/Microsoft.AspNetCore.Mvc.Core/project.json
index 5db836e30e..fd09312e8c 100644
--- a/src/Microsoft.AspNetCore.Mvc.Core/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.Core/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Authorization": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.Cors/project.json b/src/Microsoft.AspNetCore.Mvc.Cors/project.json
index 51c371462e..ab48c315b4 100644
--- a/src/Microsoft.AspNetCore.Mvc.Cors/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.Cors/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Cors": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.DataAnnotations/Internal/DataAnnotationsClientModelValidatorProvider.cs b/src/Microsoft.AspNetCore.Mvc.DataAnnotations/Internal/DataAnnotationsClientModelValidatorProvider.cs
index f68f8a120f..5c1dc946f4 100644
--- a/src/Microsoft.AspNetCore.Mvc.DataAnnotations/Internal/DataAnnotationsClientModelValidatorProvider.cs
+++ b/src/Microsoft.AspNetCore.Mvc.DataAnnotations/Internal/DataAnnotationsClientModelValidatorProvider.cs
@@ -15,7 +15,7 @@ namespace Microsoft.AspNetCore.Mvc.DataAnnotations.Internal
     /// for attributes which derive from <see cref="ValidationAttribute"/>. It also provides
     /// a validator for types which implement <see cref="IClientModelValidator"/>.
     /// The logic to support <see cref="IClientModelValidator"/>
-    /// is implemented in <see cref="ValidationAttributeAdapter{}"/>.
+    /// is implemented in <see cref="ValidationAttributeAdapter{TAttribute}"/>.
     /// </summary>
     public class DataAnnotationsClientModelValidatorProvider : IClientModelValidatorProvider
     {
@@ -26,6 +26,8 @@ namespace Microsoft.AspNetCore.Mvc.DataAnnotations.Internal
         /// <summary>
         /// Create a new instance of <see cref="DataAnnotationsClientModelValidatorProvider"/>.
         /// </summary>
+        /// <param name="validationAttributeAdapterProvider">The <see cref="IValidationAttributeAdapterProvider"/>
+        /// that supplies <see cref="IAttributeAdapter"/>s.</param>
         /// <param name="options">The <see cref="IOptions{MvcDataAnnotationsLocalizationOptions}"/>.</param>
         /// <param name="stringLocalizerFactory">The <see cref="IStringLocalizerFactory"/>.</param>
         public DataAnnotationsClientModelValidatorProvider(
diff --git a/src/Microsoft.AspNetCore.Mvc.DataAnnotations/Internal/DataAnnotationsModelValidatorProvider.cs b/src/Microsoft.AspNetCore.Mvc.DataAnnotations/Internal/DataAnnotationsModelValidatorProvider.cs
index 2444b81938..c745bbf100 100644
--- a/src/Microsoft.AspNetCore.Mvc.DataAnnotations/Internal/DataAnnotationsModelValidatorProvider.cs
+++ b/src/Microsoft.AspNetCore.Mvc.DataAnnotations/Internal/DataAnnotationsModelValidatorProvider.cs
@@ -26,6 +26,8 @@ namespace Microsoft.AspNetCore.Mvc.DataAnnotations.Internal
         /// <summary>
         /// Create a new instance of <see cref="DataAnnotationsModelValidatorProvider"/>.
         /// </summary>
+        /// <param name="validationAttributeAdapterProvider">The <see cref="IValidationAttributeAdapterProvider"/>
+        /// that supplies <see cref="IAttributeAdapter"/>s.</param>
         /// <param name="options">The <see cref="IOptions{MvcDataAnnotationsLocalizationOptions}"/>.</param>
         /// <param name="stringLocalizerFactory">The <see cref="IStringLocalizerFactory"/>.</param>
         /// <remarks><paramref name="options"/> and <paramref name="stringLocalizerFactory"/>
diff --git a/src/Microsoft.AspNetCore.Mvc.DataAnnotations/project.json b/src/Microsoft.AspNetCore.Mvc.DataAnnotations/project.json
index 19f7efd6ca..73e9e3b3df 100644
--- a/src/Microsoft.AspNetCore.Mvc.DataAnnotations/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.DataAnnotations/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Mvc.Core": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.Formatters.Json/Internal/JsonResultExecutor.cs b/src/Microsoft.AspNetCore.Mvc.Formatters.Json/Internal/JsonResultExecutor.cs
index 5a00321635..79c9ee851d 100644
--- a/src/Microsoft.AspNetCore.Mvc.Formatters.Json/Internal/JsonResultExecutor.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Formatters.Json/Internal/JsonResultExecutor.cs
@@ -31,6 +31,7 @@ namespace Microsoft.AspNetCore.Mvc.Formatters.Json.Internal
         /// <param name="writerFactory">The <see cref="IHttpResponseStreamWriterFactory"/>.</param>
         /// <param name="logger">The <see cref="ILogger{JsonResultExecutor}"/>.</param>
         /// <param name="options">The <see cref="IOptions{MvcJsonOptions}"/>.</param>
+        /// <param name="charPool">The <see cref="ArrayPool{Char}"/> for creating <see cref="T:char[]"/> buffers.</param>
         public JsonResultExecutor(
             IHttpResponseStreamWriterFactory writerFactory, 
             ILogger<JsonResultExecutor> logger, 
diff --git a/src/Microsoft.AspNetCore.Mvc.Formatters.Json/JsonInputFormatter.cs b/src/Microsoft.AspNetCore.Mvc.Formatters.Json/JsonInputFormatter.cs
index 45cb73608a..41415d5310 100644
--- a/src/Microsoft.AspNetCore.Mvc.Formatters.Json/JsonInputFormatter.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Formatters.Json/JsonInputFormatter.cs
@@ -52,7 +52,7 @@ namespace Microsoft.AspNetCore.Mvc.Formatters
         /// </summary>
         /// <param name="logger">The <see cref="ILogger"/>.</param>
         /// <param name="serializerSettings">The <see cref="JsonSerializerSettings"/>.</param>
-        /// <param name="charPool">The <see cref="ArrayPool{char}"/>.</param>
+        /// <param name="charPool">The <see cref="ArrayPool{Char}"/>.</param>
         /// <param name="objectPoolProvider">The <see cref="ObjectPoolProvider"/>.</param>
         public JsonInputFormatter(
             ILogger logger,
diff --git a/src/Microsoft.AspNetCore.Mvc.Formatters.Json/project.json b/src/Microsoft.AspNetCore.Mvc.Formatters.Json/project.json
index 712f305b71..bc3793a212 100644
--- a/src/Microsoft.AspNetCore.Mvc.Formatters.Json/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.Formatters.Json/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.JsonPatch": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.Formatters.Xml/project.json b/src/Microsoft.AspNetCore.Mvc.Formatters.Xml/project.json
index d266df7d19..032dad4259 100644
--- a/src/Microsoft.AspNetCore.Mvc.Formatters.Xml/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.Formatters.Xml/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Mvc.Core": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.Localization/HtmlLocalizerExtensions.cs b/src/Microsoft.AspNetCore.Mvc.Localization/HtmlLocalizerExtensions.cs
index 4cb57175cc..30a463c9f6 100644
--- a/src/Microsoft.AspNetCore.Mvc.Localization/HtmlLocalizerExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Localization/HtmlLocalizerExtensions.cs
@@ -15,7 +15,8 @@ namespace Microsoft.AspNetCore.Mvc.Localization
         /// <summary>
         /// Gets the <see cref="LocalizedHtmlString"/> resource for a specific name.
         /// </summary>
-        /// <param name="key">The key to use.</param>
+        /// <param name="htmlLocalizer">The <see cref="IHtmlLocalizer"/>.</param>
+        /// <param name="name">The key to use.</param>
         /// <returns>The <see cref="LocalizedHtmlString"/> resource.</returns>
         public static LocalizedHtmlString GetHtml(this IHtmlLocalizer htmlLocalizer, string name)
         {
@@ -35,7 +36,8 @@ namespace Microsoft.AspNetCore.Mvc.Localization
         /// <summary>
         /// Gets the <see cref="LocalizedHtmlString"/> resource for a specific name.
         /// </summary>
-        /// <param name="key">The key to use.</param>
+        /// <param name="htmlLocalizer">The <see cref="IHtmlLocalizer"/>.</param>
+        /// <param name="name">The key to use.</param>
         /// <param name="arguments">The values to format the string with.</param>
         /// <returns>The <see cref="LocalizedHtmlString"/> resource.</returns>
         public static LocalizedHtmlString GetHtml(this IHtmlLocalizer htmlLocalizer, string name, params object[] arguments)
diff --git a/src/Microsoft.AspNetCore.Mvc.Localization/IHtmlLocalizer.cs b/src/Microsoft.AspNetCore.Mvc.Localization/IHtmlLocalizer.cs
index 54716ed8a2..0129486bcf 100644
--- a/src/Microsoft.AspNetCore.Mvc.Localization/IHtmlLocalizer.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Localization/IHtmlLocalizer.cs
@@ -32,7 +32,6 @@ namespace Microsoft.AspNetCore.Mvc.Localization
         /// <summary>
         /// Gets the string resource with the given name.
         /// </summary>
-        /// <param name="stringLocalizer">The <see cref="IStringLocalizer"/>.</param>
         /// <param name="name">The name of the string resource.</param>
         /// <returns>The string resource as a <see cref="LocalizedString"/>.</returns>
         LocalizedString GetString(string name);
@@ -40,7 +39,6 @@ namespace Microsoft.AspNetCore.Mvc.Localization
         /// <summary>
         /// Gets the string resource with the given name and formatted with the supplied arguments.
         /// </summary>
-        /// <param name="stringLocalizer">The <see cref="IStringLocalizer"/>.</param>
         /// <param name="name">The name of the string resource.</param>
         /// <param name="arguments">The values to format the string with.</param>
         /// <returns>The formatted string resource as a <see cref="LocalizedString"/>.</returns>
diff --git a/src/Microsoft.AspNetCore.Mvc.Localization/project.json b/src/Microsoft.AspNetCore.Mvc.Localization/project.json
index d6ed7266bb..cb5ed771aa 100644
--- a/src/Microsoft.AspNetCore.Mvc.Localization/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.Localization/project.json
@@ -3,7 +3,9 @@
   "version": "1.0.0-*",
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Localization": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.Razor.Host/Directives/IChunkMerger.cs b/src/Microsoft.AspNetCore.Mvc.Razor.Host/Directives/IChunkMerger.cs
index eb4a7c98d1..d171efacf6 100644
--- a/src/Microsoft.AspNetCore.Mvc.Razor.Host/Directives/IChunkMerger.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Razor.Host/Directives/IChunkMerger.cs
@@ -20,7 +20,7 @@ namespace Microsoft.AspNetCore.Mvc.Razor.Directives
         /// <summary>
         /// Merges an inherited <see cref="Chunk"/> into the <see cref="ChunkTree"/>.
         /// </summary>
-        /// <param name="ChunkTree">The <see cref="ChunkTree"/> to merge into.</param>
+        /// <param name="chunkTree">The <see cref="ChunkTree"/> to merge into.</param>
         /// <param name="inheritedChunks">The <see cref="IReadOnlyList{Chunk}"/>s to merge.</param>
         void MergeInheritedChunks(ChunkTree chunkTree, IReadOnlyList<Chunk> inheritedChunks);
     }
diff --git a/src/Microsoft.AspNetCore.Mvc.Razor.Host/MvcRazorParser.cs b/src/Microsoft.AspNetCore.Mvc.Razor.Host/MvcRazorParser.cs
index 3f418bde4e..086f4ba2c1 100644
--- a/src/Microsoft.AspNetCore.Mvc.Razor.Host/MvcRazorParser.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Razor.Host/MvcRazorParser.cs
@@ -31,6 +31,7 @@ namespace Microsoft.AspNetCore.Mvc.Razor
         /// from parsed pages from _ViewImports files.</param>
         /// <param name="defaultInheritedChunks">The <see cref="IReadOnlyList{Chunk}"/> inherited by
         /// default by all Razor pages in the application.</param>
+        /// <param name="modelExpressionTypeName">The full name of the model expression <see cref="Type"/>.</param>
         public MvcRazorParser(
             RazorParser parser,
             IReadOnlyList<ChunkTree> inheritedChunkTrees,
diff --git a/src/Microsoft.AspNetCore.Mvc.Razor.Host/project.json b/src/Microsoft.AspNetCore.Mvc.Razor.Host/project.json
index 0aed69a767..5a211fe9d1 100644
--- a/src/Microsoft.AspNetCore.Mvc.Razor.Host/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.Razor.Host/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Razor.Runtime": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.Razor/Internal/RazorCompilationService.cs b/src/Microsoft.AspNetCore.Mvc.Razor/Internal/RazorCompilationService.cs
index b707707c55..f30af08729 100644
--- a/src/Microsoft.AspNetCore.Mvc.Razor/Internal/RazorCompilationService.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Razor/Internal/RazorCompilationService.cs
@@ -32,6 +32,7 @@ namespace Microsoft.AspNetCore.Mvc.Razor.Internal
         /// <param name="compilationService">The <see cref="ICompilationService"/> to compile generated code.</param>
         /// <param name="razorHost">The <see cref="IMvcRazorHost"/> to generate code from Razor files.</param>
         /// <param name="fileProviderAccessor">The <see cref="IRazorViewEngineFileProviderAccessor"/>.</param>
+        /// <param name="loggerFactory">The <see cref="ILoggerFactory"/>.</param>
         public RazorCompilationService(
             ICompilationService compilationService,
             IMvcRazorHost razorHost,
diff --git a/src/Microsoft.AspNetCore.Mvc.Razor/RazorPage.cs b/src/Microsoft.AspNetCore.Mvc.Razor/RazorPage.cs
index 4def724109..c6ca34b420 100644
--- a/src/Microsoft.AspNetCore.Mvc.Razor/RazorPage.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Razor/RazorPage.cs
@@ -761,6 +761,8 @@ namespace Microsoft.AspNetCore.Mvc.Razor
         /// In layout pages, asynchronously renders the content of the section named <paramref name="name"/>.
         /// </summary>
         /// <param name="name">The section to render.</param>
+        /// <param name="required">Indicates the <paramref name="name"/> section must be registered
+        /// (using <c>@section</c>) in the page.</param>
         /// <returns>A <see cref="Task{HtmlString}"/> that on completion returns <see cref="HtmlString.Empty"/> that
         /// allows the <see cref="Write(object)"/> call to succeed.</returns>
         /// <remarks>The method writes to the <see cref="Output"/> and the value returned is a token
@@ -814,7 +816,7 @@ namespace Microsoft.AspNetCore.Mvc.Razor
         }
 
         /// <summary>
-        /// Invokes <see cref="TextWriter.FlushAsync"/> on <see cref="Output"/> and <see cref="Stream.FlushAsync"/>
+        /// Invokes <see cref="TextWriter.FlushAsync"/> on <see cref="Output"/> and <see cref="M:Stream.FlushAsync"/>
         /// on the response stream, writing out any buffered content to the <see cref="HttpResponse.Body"/>.
         /// </summary>
         /// <returns>A <see cref="Task{HtmlString}"/> that represents the asynchronous flush operation and on
diff --git a/src/Microsoft.AspNetCore.Mvc.Razor/TagHelpers/UrlResolutionTagHelper.cs b/src/Microsoft.AspNetCore.Mvc.Razor/TagHelpers/UrlResolutionTagHelper.cs
index c35d09a47a..75c0f7dbc4 100644
--- a/src/Microsoft.AspNetCore.Mvc.Razor/TagHelpers/UrlResolutionTagHelper.cs
+++ b/src/Microsoft.AspNetCore.Mvc.Razor/TagHelpers/UrlResolutionTagHelper.cs
@@ -87,7 +87,7 @@ namespace Microsoft.AspNetCore.Mvc.Razor.TagHelpers
         /// <summary>
         /// Creates a new <see cref="UrlResolutionTagHelper"/>.
         /// </summary>
-        /// <param name="urlHelper">The <see cref="IUrlHelperFactory"/>.</param>
+        /// <param name="urlHelperFactory">The <see cref="IUrlHelperFactory"/>.</param>
         /// <param name="htmlEncoder">The <see cref="HtmlEncoder"/>.</param>
         public UrlResolutionTagHelper(IUrlHelperFactory urlHelperFactory, HtmlEncoder htmlEncoder)
         {
diff --git a/src/Microsoft.AspNetCore.Mvc.Razor/project.json b/src/Microsoft.AspNetCore.Mvc.Razor/project.json
index a0b4ce0eda..75a48f0195 100644
--- a/src/Microsoft.AspNetCore.Mvc.Razor/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.Razor/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Mvc.Razor.Host": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.TagHelpers/CacheTagHelper.cs b/src/Microsoft.AspNetCore.Mvc.TagHelpers/CacheTagHelper.cs
index ddc607c964..590bab435b 100644
--- a/src/Microsoft.AspNetCore.Mvc.TagHelpers/CacheTagHelper.cs
+++ b/src/Microsoft.AspNetCore.Mvc.TagHelpers/CacheTagHelper.cs
@@ -44,6 +44,7 @@ namespace Microsoft.AspNetCore.Mvc.TagHelpers
         /// Creates a new <see cref="CacheTagHelper"/>.
         /// </summary>
         /// <param name="memoryCache">The <see cref="IMemoryCache"/>.</param>
+        /// <param name="htmlEncoder">The <see cref="HtmlEncoder"/> to use.</param>
         public CacheTagHelper(IMemoryCache memoryCache, HtmlEncoder htmlEncoder)
         {
             MemoryCache = memoryCache;
diff --git a/src/Microsoft.AspNetCore.Mvc.TagHelpers/ImageTagHelper.cs b/src/Microsoft.AspNetCore.Mvc.TagHelpers/ImageTagHelper.cs
index 498551204d..b873f8a17d 100644
--- a/src/Microsoft.AspNetCore.Mvc.TagHelpers/ImageTagHelper.cs
+++ b/src/Microsoft.AspNetCore.Mvc.TagHelpers/ImageTagHelper.cs
@@ -38,6 +38,7 @@ namespace Microsoft.AspNetCore.Mvc.TagHelpers
         /// </summary>
         /// <param name="hostingEnvironment">The <see cref="IHostingEnvironment"/>.</param>
         /// <param name="cache">The <see cref="IMemoryCache"/>.</param>
+        /// <param name="htmlEncoder">The <see cref="HtmlEncoder"/> to use.</param>
         /// <param name="urlHelperFactory">The <see cref="IUrlHelperFactory"/>.</param>
         public ImageTagHelper(
             IHostingEnvironment hostingEnvironment,
diff --git a/src/Microsoft.AspNetCore.Mvc.TagHelpers/project.json b/src/Microsoft.AspNetCore.Mvc.TagHelpers/project.json
index ad259fff81..d205ae11b0 100644
--- a/src/Microsoft.AspNetCore.Mvc.TagHelpers/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.TagHelpers/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Mvc.Razor": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Controller.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Controller.cs
index b839094e61..907fa4b4aa 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Controller.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Controller.cs
@@ -244,7 +244,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <see cref="ViewComponents.ViewComponentDescriptor.FullName"/>.</param>
         /// <param name="arguments">
         /// An <see cref="object"/> with properties representing arguments to be passed to the invoked view component
-        /// method. Alternatively, an <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance
+        /// method. Alternatively, an <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance
         /// containing the invocation arguments.
         /// </param>
         /// <returns>The created <see cref="ViewComponentResult"/> object for the response.</returns>
@@ -267,7 +267,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <param name="componentType">The view component <see cref="Type"/>.</param>
         /// <param name="arguments">
         /// An <see cref="object"/> with properties representing arguments to be passed to the invoked view component
-        /// method. Alternatively, an <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance
+        /// method. Alternatively, an <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance
         /// containing the invocation arguments.
         /// </param>
         /// <returns>The created <see cref="ViewComponentResult"/> object for the response.</returns>
@@ -383,7 +383,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <summary>
         /// Releases all resources currently used by this <see cref="Controller"/> instance.
         /// </summary>
-        /// <param name="disposing"><c>true</c> if this method is being invoked by the <see cref="Dispose"/> method,
+        /// <param name="disposing"><c>true</c> if this method is being invoked by the <see cref="Dispose()"/> method,
         /// otherwise <c>false</c>.</param>
         protected virtual void Dispose(bool disposing)
         {
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/IViewComponentHelper.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/IViewComponentHelper.cs
index 6c5bcb91da..60c2552ebc 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/IViewComponentHelper.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/IViewComponentHelper.cs
@@ -18,7 +18,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <param name="name">The name of the view component.</param>
         /// <param name="arguments">
         /// An <see cref="object"/> with properties representing arguments to be passed to the invoked view component
-        /// method. Alternatively, an <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance
+        /// method. Alternatively, an <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance
         /// containing the invocation arguments.
         /// </param>
         /// <returns>A <see cref="Task"/> that on completion returns the rendered <see cref="IHtmlContent" />.
@@ -31,7 +31,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <param name="componentType">The view component <see cref="Type"/>.</param>
         /// <param name="arguments">
         /// An <see cref="object"/> with properties representing arguments to be passed to the invoked view component
-        /// method. Alternatively, an <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance
+        /// method. Alternatively, an <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance
         /// containing the invocation arguments.
         /// </param>
         /// <returns>A <see cref="Task"/> that on completion returns the rendered <see cref="IHtmlContent" />.
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/IViewBufferScope.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/IViewBufferScope.cs
index 6d90277737..2250771fb4 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/IViewBufferScope.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/IViewBufferScope.cs
@@ -6,20 +6,20 @@ using System.IO;
 namespace Microsoft.AspNetCore.Mvc.ViewFeatures.Internal
 {
     /// <summary>
-    /// Creates and manages the lifetime of <see cref="ViewBufferValue[]"/> instances.
+    /// Creates and manages the lifetime of <see cref="T:ViewBufferValue[]"/> instances.
     /// </summary>
     public interface IViewBufferScope
     {
         /// <summary>
-        /// Gets a <see cref="ViewBufferValue[]"/>.
+        /// Gets a <see cref="T:ViewBufferValue[]"/>.
         /// </summary>
-        /// <returns>The <see cref="ViewBufferValue[]"/>.</returns>
+        /// <returns>The <see cref="T:ViewBufferValue[]"/>.</returns>
         ViewBufferValue[] GetSegment();
 
         /// <summary>
-        /// Returns a <see cref="ViewBufferValue[]"/> that can be reused.
+        /// Returns a <see cref="T:ViewBufferValue[]"/> that can be reused.
         /// </summary>
-        /// <param name="segment">The <see cref="ViewBufferValue[]"/>.</param>
+        /// <param name="segment">The <see cref="T:ViewBufferValue[]"/>.</param>
         void ReturnSegment(ViewBufferValue[] segment);
 
         /// <summary>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/MemoryPoolViewBufferScope.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/MemoryPoolViewBufferScope.cs
index 362f5ab72e..d0613afe91 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/MemoryPoolViewBufferScope.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/MemoryPoolViewBufferScope.cs
@@ -27,7 +27,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures.Internal
         /// The <see cref="ArrayPool{ViewBufferValue}"/> for creating <see cref="ViewBufferValue"/> instances.
         /// </param>
         /// <param name="charPool">
-        /// The <see cref="ArrayPool{char}"/> for creating <see cref="ViewBufferTextWriter"/> instances.
+        /// The <see cref="ArrayPool{Char}"/> for creating <see cref="ViewBufferTextWriter"/> instances.
         /// </param>
         public MemoryPoolViewBufferScope(ArrayPool<ViewBufferValue> viewBufferPool, ArrayPool<char> charPool)
         {
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/ViewBufferValue.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/ViewBufferValue.cs
index 12b32e8caf..89c9581848 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/ViewBufferValue.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Internal/ViewBufferValue.cs
@@ -26,7 +26,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures.Internal
         /// <summary>
         /// Initializes a new instance of <see cref="ViewBufferValue"/> with a <see cref="IHtmlContent"/> value.
         /// </summary>
-        /// <param name="value">The <see cref="IHtmlContent"/>.</param>
+        /// <param name="content">The <see cref="IHtmlContent"/>.</param>
         public ViewBufferValue(IHtmlContent content)
         {
             Value = content;
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ModelStateDictionaryExtensions.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ModelStateDictionaryExtensions.cs
index 3cafb812c3..9c78d12614 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ModelStateDictionaryExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ModelStateDictionaryExtensions.cs
@@ -52,6 +52,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         /// <param name="modelState">The <see cref="ModelStateDictionary"/> instance this method extends.</param>
         /// <param name="expression">An expression to be evaluated against an item in the current model.</param>
         /// <param name="exception">The <see cref="Exception"/> to add.</param>
+        /// <param name="metadata">The <see cref="ModelMetadata"/> associated with the model.</param>
         public static void AddModelError<TModel>(
             this ModelStateDictionary modelState,
             Expression<Func<TModel, object>> expression,
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperDisplayExtensions.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperDisplayExtensions.cs
index 58552939d4..c48dcf03fa 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperDisplayExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperDisplayExtensions.cs
@@ -57,7 +57,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <see cref="object"/> that contains the properties to display.
         /// </param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -144,7 +144,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="templateName">The name of the template used to create the HTML markup.</param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -273,7 +273,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="htmlHelper">The <see cref="IHtmlHelper{TModel}"/> instance this method extends.</param>
         /// <param name="expression">An expression to be evaluated against the current model.</param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -364,7 +364,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="expression">An expression to be evaluated against the current model.</param>
         /// <param name="templateName">The name of the template used to create the HTML markup.</param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -488,7 +488,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </summary>
         /// <param name="htmlHelper">The <see cref="IHtmlHelper"/> instance this method extends.</param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -556,7 +556,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="htmlHelper">The <see cref="IHtmlHelper"/> instance this method extends.</param>
         /// <param name="templateName">The name of the template used to create the HTML markup.</param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -639,7 +639,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// properties that have the same name.
         /// </param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperDisplayNameExtensions.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperDisplayNameExtensions.cs
index 536f071f7c..7280159990 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperDisplayNameExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperDisplayNameExtensions.cs
@@ -32,7 +32,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// if the current model represents a collection.
         /// </summary>
         /// <param name="htmlHelper">
-        /// The <see cref="IHtmlHelper{IEnumerable{TModelItem}}"/> instance this method extends.
+        /// The <see cref="IHtmlHelper{T}"/> of <see cref="IEnumerable{TModelItem}"/> instance this method extends.
         /// </param>
         /// <param name="expression">An expression to be evaluated against an item in the current model.</param>
         /// <typeparam name="TModelItem">The type of items in the model collection.</typeparam>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperEditorExtensions.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperEditorExtensions.cs
index bc21542e6c..645d75789b 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperEditorExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperEditorExtensions.cs
@@ -57,7 +57,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <see cref="object"/> that contains the properties to edit.
         /// </param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -141,7 +141,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="templateName">The name of the template used to create the HTML markup.</param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -266,7 +266,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="htmlHelper">The <see cref="IHtmlHelper{TModel}"/> instance this method extends.</param>
         /// <param name="expression">An expression to be evaluated against the current model.</param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -353,7 +353,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="expression">An expression to be evaluated against the current model.</param>
         /// <param name="templateName">The name of the template that is used to create the HTML markup.</param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -446,7 +446,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <remarks>
         /// <para>
         /// For example the default <see cref="object"/> editor template includes &lt;label&gt; and &lt;input&gt;
-        /// elements for each property in the <paramref name="expression"/> result.
+        /// elements for each property in the current model.
         /// </para>
         /// <para>
         /// Custom templates are found under a <c>EditorTemplates</c> folder. The folder name is case-sensitive on
@@ -473,7 +473,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </summary>
         /// <param name="htmlHelper">The <see cref="IHtmlHelper"/> instance this method extends.</param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -541,7 +541,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="htmlHelper">The <see cref="IHtmlHelper"/> instance this method extends.</param>
         /// <param name="templateName">The name of the template used to create the HTML markup.</param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
@@ -624,7 +624,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// properties that have the same name.
         /// </param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{string, object}"/>
+        /// An anonymous <see cref="object"/> or <see cref="System.Collections.Generic.IDictionary{String, Object}"/>
         /// that can contain additional view data that will be merged into the
         /// <see cref="ViewFeatures.ViewDataDictionary{TModel}"/> instance created for the template.
         /// </param>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperFormExtensions.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperFormExtensions.cs
index 17e97801c7..3a354679c4 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperFormExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperFormExtensions.cs
@@ -46,8 +46,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="antiforgery">
         /// If <c>true</c>, &lt;form&gt; elements will include an antiforgery token.
         /// If <c>false</c>, suppresses the generation an &lt;input&gt; of type "hidden" with an antiforgery token.
-        /// If <c>null</c>, &lt;form&gt; elements will include an antiforgery token only if
-        /// <paramref name="method"/> is not <see cref="FormMethod.Get"/>.
+        /// If <c>null</c>, &lt;form&gt; elements will include an antiforgery token.
         /// </param>
         /// <returns>
         /// An <see cref="MvcForm"/> instance which renders the &lt;/form&gt; end tag when disposed.
@@ -108,7 +107,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>
@@ -150,7 +149,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>
@@ -188,7 +187,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <returns>
@@ -256,7 +255,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <returns>
@@ -330,7 +329,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
@@ -371,7 +370,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>
@@ -410,7 +409,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <returns>
@@ -443,14 +442,13 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <param name="antiforgery">
         /// If <c>true</c>, &lt;form&gt; elements will include an antiforgery token.
         /// If <c>false</c>, suppresses the generation an &lt;input&gt; of type "hidden" with an antiforgery token.
-        /// If <c>null</c>, &lt;form&gt; elements will include an antiforgery token only if
-        /// <paramref name="method"/> is not <see cref="FormMethod.Get"/>.
+        /// If <c>null</c>, &lt;form&gt; elements will include an antiforgery token.
         /// </param>
         /// <returns>
         /// An <see cref="MvcForm"/> instance which renders the &lt;/form&gt; end tag when disposed.
@@ -509,8 +507,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="antiforgery">
         /// If <c>true</c>, &lt;form&gt; elements will include an antiforgery token.
         /// If <c>false</c>, suppresses the generation an &lt;input&gt; of type "hidden" with an antiforgery token.
-        /// If <c>null</c>, &lt;form&gt; elements will include an antiforgery token only if
-        /// <paramref name="method"/> is not <see cref="FormMethod.Get"/>.
+        /// If <c>null</c>, &lt;form&gt; elements will include an antiforgery token.
         /// </param>
         /// <returns>
         /// An <see cref="MvcForm"/> instance which renders the &lt;/form&gt; end tag when disposed.
@@ -543,7 +540,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <returns>
@@ -611,7 +608,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
@@ -649,7 +646,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperInputExtensions.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperInputExtensions.cs
index 09ce8c314e..980f69ca2c 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperInputExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperInputExtensions.cs
@@ -117,7 +117,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="expression">Expression name, relative to the current model.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the checkbox element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; elements.</returns>
@@ -469,8 +469,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <para>
         /// In all but the default case, includes a "checked" attribute with
         /// value "checked" if the <see cref="string"/> values is equal to a converted <see cref="string"/> for
-        /// <paramref name="value"/> or <paramref name="isChecked"/> is <c>true</c> (for that case); does not include
-        /// the attribute otherwise.
+        /// <paramref name="value"/>; does not include the attribute otherwise.
         /// </para>
         /// </remarks>
         public static IHtmlContent RadioButton(
@@ -497,7 +496,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; element.</returns>
@@ -535,8 +534,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <para>
         /// In all but the <paramref name="htmlAttributes"/> and default cases, includes a "checked" attribute with
         /// value "checked" if the <see cref="string"/> values is equal to a converted <see cref="string"/> for
-        /// <paramref name="value"/> or <paramref name="isChecked"/> is <c>true</c> (for that case); does not include
-        /// the attribute otherwise.
+        /// <paramref name="value"/>; does not include the attribute otherwise.
         /// </para>
         /// </remarks>
         public static IHtmlContent RadioButton(
@@ -828,7 +826,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="value">If non-<c>null</c>, value to include in the element.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; element.</returns>
@@ -974,7 +972,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="expression">An expression to be evaluated against the current model.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <typeparam name="TModel">The type of the model.</typeparam>
@@ -1067,7 +1065,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="expression">Expression name, relative to the current model.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;textarea&gt; element.</returns>
@@ -1163,7 +1161,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="value">If non-<c>null</c>, value to include in the element.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;textarea&gt; element.</returns>
@@ -1257,7 +1255,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="expression">An expression to be evaluated against the current model.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <typeparam name="TModel">The type of the model.</typeparam>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperLabelExtensions.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperLabelExtensions.cs
index cffda91e8e..f28b30ef3f 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperLabelExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperLabelExtensions.cs
@@ -104,7 +104,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="expression">An expression to be evaluated against the current model.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <typeparam name="TModel">The type of the model.</typeparam>
@@ -165,7 +165,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="htmlHelper">The <see cref="IHtmlHelper"/> instance this method extends.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;label&gt; element.</returns>
@@ -186,7 +186,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="labelText">The inner text of the element.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;label&gt; element.</returns>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperLinkExtensions.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperLinkExtensions.cs
index 506745dd07..aab8bc53fa 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperLinkExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperLinkExtensions.cs
@@ -54,7 +54,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the anchor element.</returns>
@@ -95,12 +95,12 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the anchor element.</returns>
@@ -178,7 +178,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the anchor element.</returns>
@@ -221,12 +221,12 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the anchor element.</returns>
@@ -268,7 +268,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the anchor element.</returns>
@@ -339,7 +339,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the anchor element.</returns>
@@ -378,12 +378,12 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the anchor element.</returns>
@@ -423,12 +423,12 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the route
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the route
         /// parameters.
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the anchor element.</returns>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperSelectExtensions.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperSelectExtensions.cs
index 4233634883..c6f42f787e 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperSelectExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperSelectExtensions.cs
@@ -107,7 +107,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the &lt;select&gt; element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;select&gt; element.</returns>
         /// <remarks>
@@ -210,7 +210,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the &lt;select&gt; element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <typeparam name="TModel">The type of the model.</typeparam>
         /// <typeparam name="TResult">The type of the <paramref name="expression"/> result.</typeparam>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperValidationExtensions.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperValidationExtensions.cs
index 8cfafff1a7..9b5da1e9dd 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperValidationExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/HtmlHelperValidationExtensions.cs
@@ -75,7 +75,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the
         /// (<see cref="ViewContext.ValidationMessageElement"/>) element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>
@@ -146,7 +146,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the
         /// (<see cref="ViewContext.ValidationMessageElement"/>) element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <returns>
@@ -249,7 +249,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the
         /// (<see cref="ViewContext.ValidationMessageElement"/>) element. Alternatively, an
-        /// <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing the HTML
+        /// <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing the HTML
         /// attributes.
         /// </param>
         /// <typeparam name="TModel">The type of the model.</typeparam>
@@ -461,7 +461,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="message">The message to display with the validation summary.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the topmost (&lt;div&gt;) element.
-        /// Alternatively, an <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing
+        /// Alternatively, an <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing
         /// the HTML attributes.
         /// </param>
         /// <returns>
@@ -495,7 +495,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="message">The message to display with the validation summary.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the topmost (&lt;div&gt;) element.
-        /// Alternatively, an <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing
+        /// Alternatively, an <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing
         /// the HTML attributes.
         /// </param>
         /// <param name="tag">
@@ -572,7 +572,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="message">The message to display with the validation summary.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the topmost (&lt;div&gt;) element.
-        /// Alternatively, an <see cref="System.Collections.Generic.IDictionary{string, object}"/> instance containing
+        /// Alternatively, an <see cref="System.Collections.Generic.IDictionary{String, Object}"/> instance containing
         /// the HTML attributes.
         /// </param>
         /// <returns>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/IHtmlHelper.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/IHtmlHelper.cs
index ef8a9f24d1..1f2a88e7e8 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/IHtmlHelper.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/IHtmlHelper.cs
@@ -71,11 +71,11 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the route parameters.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the route parameters.
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the anchor element.</returns>
         IHtmlContent ActionLink(
@@ -105,7 +105,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the route parameters.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the route parameters.
         /// </param>
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
         /// <param name="antiforgery">
@@ -116,7 +116,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>
         /// An <see cref="MvcForm"/> instance which renders the &lt;/form&gt; end tag when disposed.
@@ -141,7 +141,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the route parameters.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the route parameters.
         /// </param>
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
         /// <param name="antiforgery">
@@ -152,7 +152,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>
         /// An <see cref="MvcForm"/> instance which renders the &lt;/form&gt; end tag when disposed.
@@ -175,7 +175,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="isChecked">If <c>true</c>, checkbox is initially checked.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the checkbox element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; elements.</returns>
         /// <remarks>
@@ -226,7 +226,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// properties that have the same name.
         /// </param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="IDictionary{string, object}"/> that can contain additional
+        /// An anonymous <see cref="object"/> or <see cref="IDictionary{String, Object}"/> that can contain additional
         /// view data that will be merged into the <see cref="ViewDataDictionary{TModel}"/> instance created for the
         /// template.
         /// </param>
@@ -278,7 +278,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the &lt;select&gt; element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;select&gt; element.</returns>
         /// <remarks>
@@ -307,7 +307,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// properties that have the same name.
         /// </param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="IDictionary{string, object}"/> that can contain additional
+        /// An anonymous <see cref="object"/> or <see cref="IDictionary{String, Object}"/> that can contain additional
         /// view data that will be merged into the <see cref="ViewDataDictionary{TModel}"/> instance created for the
         /// template.
         /// </param>
@@ -404,7 +404,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="value">If non-<c>null</c>, value to include in the element.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; element.</returns>
         /// <remarks>
@@ -450,7 +450,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="labelText">The inner text of the element.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;label&gt; element.</returns>
         IHtmlContent Label(string expression, string labelText, object htmlAttributes);
@@ -466,7 +466,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the &lt;select&gt; element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;select&gt; element.</returns>
         /// <remarks>
@@ -504,7 +504,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="value">If non-<c>null</c>, value to include in the element.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; element.</returns>
         /// <remarks>
@@ -538,7 +538,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; element.</returns>
         /// <remarks>
@@ -624,11 +624,11 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the route parameters.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the route parameters.
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the anchor element.</returns>
         IHtmlContent RouteLink(
@@ -649,7 +649,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="columns">Number of columns in the textarea.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;textarea&gt; element.</returns>
         /// <remarks>
@@ -690,7 +690,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; element.</returns>
         /// <remarks>
@@ -740,7 +740,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the <paramref name="tag"/> element.
-        /// Alternatively, an <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// Alternatively, an <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <param name="tag">
         /// The tag to wrap the <paramref name="message"/> in the generated HTML. Its default value is
@@ -762,7 +762,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="message">The message to display with the validation summary.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the topmost (&lt;div&gt;) element.
-        /// Alternatively, an <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// Alternatively, an <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <param name="tag">
         /// The tag to wrap the <paramref name="message"/> in the generated HTML. Its default value is
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/IHtmlHelperOfT.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/IHtmlHelperOfT.cs
index 2dd22dade4..eb1499488e 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/IHtmlHelperOfT.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/IHtmlHelperOfT.cs
@@ -27,7 +27,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="expression">An expression to be evaluated against the current model.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the checkbox element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; elements.</returns>
         /// <remarks>
@@ -68,7 +68,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// that have the same name.
         /// </param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="IDictionary{string, object}"/> that can contain additional
+        /// An anonymous <see cref="object"/> or <see cref="IDictionary{String, Object}"/> that can contain additional
         /// view data that will be merged into the <see cref="ViewDataDictionary{TModel}"/> instance created for the
         /// template.
         /// </param>
@@ -129,7 +129,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the &lt;select&gt; element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <typeparam name="TResult">The type of the <paramref name="expression"/> result.</typeparam>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;select&gt; element.</returns>
@@ -156,7 +156,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// that have the same name.
         /// </param>
         /// <param name="additionalViewData">
-        /// An anonymous <see cref="object"/> or <see cref="IDictionary{string, object}"/> that can contain additional
+        /// An anonymous <see cref="object"/> or <see cref="IDictionary{String, Object}"/> that can contain additional
         /// view data that will be merged into the <see cref="ViewDataDictionary{TModel}"/> instance created for the
         /// template.
         /// </param>
@@ -184,7 +184,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="expression">An expression to be evaluated against the current model.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <typeparam name="TResult">The type of the <paramref name="expression"/> result.</typeparam>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; element.</returns>
@@ -227,7 +227,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="labelText">The inner text of the element.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <typeparam name="TResult">The type of the <paramref name="expression"/> result.</typeparam>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;label&gt; element.</returns>
@@ -247,7 +247,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the &lt;select&gt; element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <typeparam name="TResult">The type of the <paramref name="expression"/> result.</typeparam>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;select&gt; element.</returns>
@@ -275,7 +275,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="expression">An expression to be evaluated against the current model.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <typeparam name="TResult">The type of the <paramref name="expression"/> result.</typeparam>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; element.</returns>
@@ -306,7 +306,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="value">Value to include in the element. Must not be <c>null</c>.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <typeparam name="TResult">The type of the <paramref name="expression"/> result.</typeparam>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; element.</returns>
@@ -354,7 +354,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="columns">Number of columns in the textarea.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <typeparam name="TResult">The type of the <paramref name="expression"/> result.</typeparam>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;textarea&gt; element.</returns>
@@ -391,7 +391,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <typeparam name="TResult">The type of the <paramref name="expression"/> result.</typeparam>
         /// <returns>A new <see cref="IHtmlContent"/> containing the &lt;input&gt; element.</returns>
@@ -433,7 +433,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the <paramref name="tag"/> element.
-        /// Alternatively, an <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// Alternatively, an <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <param name="tag">
         /// The tag to wrap the <paramref name="message"/> in the generated HTML. Its default value is
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/ViewComponentHelperExtensions.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/ViewComponentHelperExtensions.cs
index 635d7c6a9c..d973a8a5eb 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/ViewComponentHelperExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/ViewComponentHelperExtensions.cs
@@ -15,6 +15,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <summary>
         /// Invokes a view component with the specified <paramref name="name"/>.
         /// </summary>
+        /// <param name="helper">The <see cref="IViewComponentHelper"/>.</param>
         /// <param name="name">The name of the view component.</param>
         /// <returns>A <see cref="Task"/> that on completion returns the rendered <see cref="IHtmlContent" />.
         /// </returns>
@@ -31,6 +32,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <summary>
         /// Invokes a view component of type <paramref name="componentType" />.
         /// </summary>
+        /// <param name="helper">The <see cref="IViewComponentHelper"/>.</param>
         /// <param name="componentType">The view component <see cref="Type"/>.</param>
         /// <returns>A <see cref="Task"/> that on completion returns the rendered <see cref="IHtmlContent" />.
         /// </returns>
@@ -45,7 +47,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         }
 
         /// <summary>
-        /// Invokes a view component of type <typeparam name="TComponent"/>.
+        /// Invokes a view component of type <typeparamref name="TComponent"/>.
         /// </summary>
         /// <param name="helper">The <see cref="IViewComponentHelper"/>.</param>
         /// <param name="arguments">Arguments to be passed to the invoked view component method.</param>
@@ -63,7 +65,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         }
 
         /// <summary>
-        /// Invokes a view component of type <typeparam name="TComponent"/>.
+        /// Invokes a view component of type <typeparamref name="TComponent"/>.
         /// </summary>
         /// <param name="helper">The <see cref="IViewComponentHelper"/>.</param>
         /// <typeparam name="TComponent">The <see cref="Type"/> of the view component.</typeparam>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/ViewContext.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/ViewContext.cs
index 8e7c0e49f3..d0bc2b56ca 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/ViewContext.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/Rendering/ViewContext.cs
@@ -37,6 +37,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
         /// <param name="viewData">The <see cref="ViewDataDictionary"/>.</param>
         /// <param name="tempData">The <see cref="ITempDataDictionary"/>.</param>
         /// <param name="writer">The <see cref="TextWriter"/> to render output to.</param>
+        /// <param name="htmlHelperOptions">The <see cref="HtmlHelperOptions"/> to apply to this instance.</param>
         public ViewContext(
             ActionContext actionContext,
             IView view,
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/DefaultViewComponentInvoker.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/DefaultViewComponentInvoker.cs
index 8df665be83..bde18ca400 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/DefaultViewComponentInvoker.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/DefaultViewComponentInvoker.cs
@@ -26,7 +26,6 @@ namespace Microsoft.AspNetCore.Mvc.ViewComponents
         /// <summary>
         /// Initializes a new instance of <see cref="DefaultViewComponentInvoker"/>.
         /// </summary>
-        /// <param name="typeActivatorCache">Caches factories for instantiating view component instances.</param>
         /// <param name="viewComponentFactory">The <see cref="IViewComponentFactory"/>.</param>
         /// <param name="diagnosticSource">The <see cref="DiagnosticSource"/>.</param>
         /// <param name="logger">The <see cref="ILogger"/>.</param>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/IViewComponentActivator.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/IViewComponentActivator.cs
index 9f42143c48..92c3b4971b 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/IViewComponentActivator.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/IViewComponentActivator.cs
@@ -20,7 +20,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewComponents
         /// Releases a ViewComponent instance.
         /// </summary>
         /// <param name="context">
-        /// The <see cref="ViewComponentContext"/> associated with the <paramref name="component"/>.
+        /// The <see cref="ViewComponentContext"/> associated with the <paramref name="viewComponent"/>.
         /// </param>
         /// <param name="viewComponent">The <see cref="ViewComponent"/> to release.</param>
         void Release(ViewComponentContext context, object viewComponent);
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/ViewComponentContext.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/ViewComponentContext.cs
index 2c79723c20..e45ba82154 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/ViewComponentContext.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/ViewComponentContext.cs
@@ -34,6 +34,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewComponents
         /// The <see cref="ViewComponentContext"/> for the view component being invoked.
         /// </param>
         /// <param name="arguments">The view component arguments.</param>
+        /// <param name="htmlEncoder">The <see cref="HtmlEncoder"/> to use.</param>
         /// <param name="viewContext">The <see cref="ViewContext"/>.</param>
         /// <param name="writer">The <see cref="TextWriter"/> for writing output.</param>
         public ViewComponentContext(
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/DefaultHtmlGenerator.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/DefaultHtmlGenerator.cs
index e055e8ebbd..c104384b09 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/DefaultHtmlGenerator.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/DefaultHtmlGenerator.cs
@@ -45,8 +45,10 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// tokens.</param>
         /// <param name="optionsAccessor">The accessor for <see cref="MvcOptions"/>.</param>
         /// <param name="metadataProvider">The <see cref="IModelMetadataProvider"/>.</param>
-        /// <param name="urlHelper">The <see cref="IUrlHelper"/>.</param>
+        /// <param name="urlHelperFactory">The <see cref="IUrlHelperFactory"/>.</param>
         /// <param name="htmlEncoder">The <see cref="HtmlEncoder"/>.</param>
+        /// <param name="clientValidatorCache">The <see cref="ClientValidatorCache"/> that provides
+        /// a list of <see cref="IClientModelValidator"/>s.</param>
         public DefaultHtmlGenerator(
             IAntiforgery antiforgery,
             IOptions<MvcViewOptions> optionsAccessor,
@@ -1077,7 +1079,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>
         /// A <see cref="TagBuilder"/> instance for the &lt;/form&gt; element.
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/HtmlHelper.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/HtmlHelper.cs
index 1d86236cc5..5d3dcb19e1 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/HtmlHelper.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/HtmlHelper.cs
@@ -163,7 +163,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// value to the dictionary. It will expose public properties from derived types as well. This is typically
         /// used with objects of an anonymous type.
         ///
-        /// If the <paramref name="value"/> is already an <see cref="IDictionary{string, object}"/> instance, then it
+        /// If the <paramref name="value"/> is already an <see cref="IDictionary{String, Object}"/> instance, then it
         /// is returned as-is.
         /// <example>
         /// <c>new { data_name="value" }</c> will translate to the entry <c>{ "data_name", "value" }</c>
@@ -181,7 +181,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// Creates a dictionary of HTML attributes from the input object,
         /// translating underscores to dashes in each public instance property.
         ///
-        /// If the object is already an <see cref="IDictionary{string, object}"/> instance, then it is
+        /// If the object is already an <see cref="IDictionary{String, Object}"/> instance, then it is
         /// returned as-is.
         /// <example>
         /// <c>new { data_name="value" }</c> will translate to the entry <c>{ "data-name", "value" }</c>
@@ -856,7 +856,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the route parameters.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the route parameters.
         /// </param>
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
         /// <param name="antiforgery">
@@ -867,7 +867,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>
         /// An <see cref="MvcForm"/> instance which renders the &lt;/form&gt; end tag when disposed.
@@ -915,7 +915,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the route parameters.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the route parameters.
         /// </param>
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
         /// <param name="antiforgery">
@@ -926,7 +926,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>
         /// An <see cref="MvcForm"/> instance which renders the &lt;/form&gt; end tag when disposed.
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/IHtmlGenerator.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/IHtmlGenerator.cs
index c129e4638a..4a46799e74 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/IHtmlGenerator.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/IHtmlGenerator.cs
@@ -35,11 +35,11 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the route parameters.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the route parameters.
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>
         /// A <see cref="TagBuilder"/> instance for the &lt;a&gt; element.
@@ -74,7 +74,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// <param name="isChecked">The initial state of the checkbox element.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>
         /// A <see cref="TagBuilder"/> instance for the &lt;input type="checkbox".../&gt; element.
@@ -107,12 +107,12 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the route parameters.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the route parameters.
         /// </param>
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>
         /// A <see cref="TagBuilder"/> instance for the &lt;/form&gt; element.
@@ -135,12 +135,12 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the route parameters.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the route parameters.
         /// </param>
         /// <param name="method">The HTTP method for processing the form, either GET or POST.</param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>
         /// A <see cref="TagBuilder"/> instance for the &lt;/form&gt; element.
@@ -189,17 +189,17 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// <param name="linkText">The text to insert inside the element.</param>
         /// <param name="routeName">The name of the route to use for link generation.</param>
         /// <param name="protocol">The protocol (scheme) for the generated link.</param>
-        /// <param name="hostname">The hostname for the generated link.</param>
+        /// <param name="hostName">The hostname for the generated link.</param>
         /// <param name="fragment">The fragment for the genrated link.</param>
         /// <param name="routeValues">
         /// An <see cref="object"/> that contains the parameters for a route. The parameters are retrieved through
         /// reflection by examining the properties of the <see cref="object"/>. This <see cref="object"/> is typically
         /// created using <see cref="object"/> initializer syntax. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the route parameters.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the route parameters.
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>
         /// A <see cref="TagBuilder"/> instance for the &lt;a&gt; element.
@@ -235,7 +235,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the &lt;select&gt; element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="TagBuilder"/> describing the &lt;select&gt; element.</returns>
         /// <remarks>
@@ -273,7 +273,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// <c>ViewContext.ViewData[expression]</c>.
         /// </param>
         /// <param name="currentValues">
-        /// An <see cref="ICollection{string}"/> containing values for &lt;option&gt; elements to select. If
+        /// An <see cref="ICollection{String}"/> containing values for &lt;option&gt; elements to select. If
         /// <c>null</c>, selects &lt;option&gt; elements based on <see cref="SelectListItem.Selected"/> values in
         /// <paramref name="selectList"/>.
         /// </param>
@@ -283,7 +283,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the &lt;select&gt; element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>A new <see cref="TagBuilder"/> describing the &lt;select&gt; element.</returns>
         /// <remarks>
@@ -354,7 +354,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// </param>
         /// <param name="htmlAttributes">
         /// An <see cref="object"/> that contains the HTML attributes for the element. Alternatively, an
-        /// <see cref="IDictionary{string, object}"/> instance containing the HTML attributes.
+        /// <see cref="IDictionary{String, Object}"/> instance containing the HTML attributes.
         /// </param>
         /// <returns>
         /// A <see cref="TagBuilder"/> containing a <paramref name="tag"/> element if the
@@ -394,7 +394,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// <returns>
         /// <para>
         /// <c>null</c> if no <paramref name="expression"/> result is found. Otherwise a
-        /// <see cref="ICollection{string}"/> containing current values for the given
+        /// <see cref="ICollection{String}"/> containing current values for the given
         /// <paramref name="expression"/>.
         /// </para>
         /// <para>
@@ -410,7 +410,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// </para>
         /// </returns>
         /// <remarks>
-        /// See <see cref="GenerateSelect"/> for information about how the return value may be used.
+        /// See <see cref="M:GenerateSelect"/> for information about how the return value may be used.
         /// </remarks>
         ICollection<string> GetCurrentValues(
             ViewContext viewContext,
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/ModelExplorer.cs b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/ModelExplorer.cs
index 74654dc956..0130bc30a0 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/ModelExplorer.cs
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/ModelExplorer.cs
@@ -55,7 +55,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// <param name="metadataProvider">The <see cref="IModelMetadataProvider"/>.</param>
         /// <param name="container">The container <see cref="ModelExplorer"/>.</param>
         /// <param name="metadata">The <see cref="ModelMetadata"/>.</param>
-        /// <param name="modelAccessor">A model accessor function. May be <c>null</c>.</param>
+        /// <param name="modelAccessor">A model accessor function..</param>
         public ModelExplorer(
             IModelMetadataProvider metadataProvider,
             ModelExplorer container,
@@ -351,7 +351,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// <summary>
         /// Gets a <see cref="ModelExplorer"/> for the provided model value and model <see cref="Type"/>.
         /// </summary>
-        /// <param name="modelType">The model <see cref="Type"/>.</param>
+        /// <param name="metadata">The <see cref="ModelMetadata"/> associated with the model.</param>
         /// <param name="model">The model value.</param>
         /// <returns>A <see cref="ModelExplorer"/>.</returns>
         /// <remarks>
@@ -379,7 +379,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// Gets a <see cref="ModelExplorer"/> for the provided model value and model <see cref="Type"/>.
         /// </summary>
         /// <param name="modelType">The model <see cref="Type"/>.</param>
-        /// <param name="model">The model value.</param>
+        /// <param name="modelAccessor">A model accessor function.</param>
         /// <returns>A <see cref="ModelExplorer"/>.</returns>
         /// <remarks>
         /// <para>
@@ -406,8 +406,8 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// <summary>
         /// Gets a <see cref="ModelExplorer"/> for the provided model value and model <see cref="Type"/>.
         /// </summary>
-        /// <param name="modelType">The model <see cref="Type"/>.</param>
-        /// <param name="model">The model value.</param>
+        /// <param name="metadata">The <see cref="ModelMetadata"/> associated with the model.</param>
+        /// <param name="modelAccessor">A model accessor function.</param>
         /// <returns>A <see cref="ModelExplorer"/>.</returns>
         /// <remarks>
         /// <para>
diff --git a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/project.json b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/project.json
index 6f0a6f6885..6f76f532c6 100644
--- a/src/Microsoft.AspNetCore.Mvc.ViewFeatures/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.ViewFeatures/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Antiforgery": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc.WebApiCompatShim/HttpRequestMessage/HttpRequestMessageExtensions.cs b/src/Microsoft.AspNetCore.Mvc.WebApiCompatShim/HttpRequestMessage/HttpRequestMessageExtensions.cs
index f14dd06a26..6877b88f74 100644
--- a/src/Microsoft.AspNetCore.Mvc.WebApiCompatShim/HttpRequestMessage/HttpRequestMessageExtensions.cs
+++ b/src/Microsoft.AspNetCore.Mvc.WebApiCompatShim/HttpRequestMessage/HttpRequestMessageExtensions.cs
@@ -296,8 +296,7 @@ namespace System.Net.Http
         /// found, this method returns a response with status 406 NotAcceptable.
         /// </summary>
         /// <remarks>
-        /// This method will use the provided <paramref name="configuration"/> or it will get the
-        /// <see cref="HttpContext"/> instance associated with <paramref name="request"/>.
+        /// This method will get the <see cref="HttpContext"/> instance associated with <paramref name="request"/>.
         /// </remarks>
         /// <typeparam name="T">The type of the value.</typeparam>
         /// <param name="request">The request.</param>
diff --git a/src/Microsoft.AspNetCore.Mvc.WebApiCompatShim/project.json b/src/Microsoft.AspNetCore.Mvc.WebApiCompatShim/project.json
index c60b8d275b..976df4e161 100644
--- a/src/Microsoft.AspNetCore.Mvc.WebApiCompatShim/project.json
+++ b/src/Microsoft.AspNetCore.Mvc.WebApiCompatShim/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Mvc.Core": "1.0.0-*",
diff --git a/src/Microsoft.AspNetCore.Mvc/project.json b/src/Microsoft.AspNetCore.Mvc/project.json
index fb7f01786a..8a3643264f 100644
--- a/src/Microsoft.AspNetCore.Mvc/project.json
+++ b/src/Microsoft.AspNetCore.Mvc/project.json
@@ -7,7 +7,9 @@
   },
   "compilationOptions": {
     "warningsAsErrors": true,
-    "keyFile": "../../tools/Key.snk"
+    "keyFile": "../../tools/Key.snk",
+    "nowarn": [ "CS1591" ],
+    "xmlDoc": true
   },
   "dependencies": {
     "Microsoft.AspNetCore.Mvc.ApiExplorer": "1.0.0-*",