House Cleaning - Spelling and grammar 🚿

benchmarks/Microsoft.AspNetCore.Mvc.Performance/ActionSelectorBenchmark.cs

@@ -21,7 +21,7 @@ namespace Microsoft.AspNetCore.Mvc.Performance
     {
         private const int Seed = 1000;
 
-        // About 35 or so plausable sounding conventional routing actions.
+        // About 35 or so plausible sounding conventional routing actions.
         //
         // We include some duplicates here, because that's what happens when you have one method that handles
         // GET and one that handles POST.

src/Microsoft.AspNetCore.Mvc.Abstractions/ActionConstraints/ActionSelectorCandidate.cs

@@ -31,7 +31,7 @@ namespace Microsoft.AspNetCore.Mvc.ActionConstraints
         }
 
         /// <summary>
-        /// The <see cref="ActionDescriptor"/> representing a candiate for selection.
+        /// The <see cref="ActionDescriptor"/> representing a candidate for selection.
         /// </summary>
         public ActionDescriptor Action { get; }
 

src/Microsoft.AspNetCore.Mvc.Abstractions/Filters/FilterDescriptor.cs

@@ -18,7 +18,7 @@ namespace Microsoft.AspNetCore.Mvc.Filters
     /// with a higher value of Scope execute later in the pipeline. See <c>Microsoft.AspNetCore.Mvc.FilterScope</c>
     /// for commonly used scopes.
     ///
-    /// For <see cref="IExceptionFilter"/> implementions, the filter runs only after an exception has occurred,
+    /// For <see cref="IExceptionFilter"/> implementations, the filter runs only after an exception has occurred,
     /// and so the observed order of execution will be opposite that of other filters.
     /// </remarks>
     public class FilterDescriptor

src/Microsoft.AspNetCore.Mvc.Abstractions/Filters/ResourceExecutedContext.cs

@@ -27,7 +27,7 @@ namespace Microsoft.AspNetCore.Mvc.Filters
 
         /// <summary>
         /// Gets or sets a value which indicates whether or not execution was canceled by a resource filter.
-        /// If true, then a resource filter short-circuted execution by setting
+        /// If true, then a resource filter short-circuited execution by setting
         /// <see cref="ResourceExecutingContext.Result"/>.
         /// </summary>
         public virtual bool Canceled { get; set; }

src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/CompositeBindingSource.cs

@@ -9,7 +9,7 @@ using Microsoft.AspNetCore.Mvc.Abstractions;
 namespace Microsoft.AspNetCore.Mvc.ModelBinding
 {
     /// <summary>
-    /// A <see cref="BindingSources"/> which can repesent multiple value-provider data sources.
+    /// A <see cref="BindingSources"/> which can represent multiple value-provider data sources.
     /// </summary>
     public class CompositeBindingSource : BindingSource
     {

src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/IBindingSourceMetadata.cs

@@ -4,7 +4,7 @@
 namespace Microsoft.AspNetCore.Mvc.ModelBinding
 {
     /// <summary>
-    /// Metadata which specificies the data source for model binding.
+    /// Metadata which specifies the data source for model binding.
     /// </summary>
     public interface IBindingSourceMetadata
     {

src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/Metadata/ModelMetadataIdentity.cs

@@ -66,7 +66,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding.Metadata
         }
 
         /// <summary>
-        /// Gets the <see cref="Type"/> defining the model property respresented by the current
+        /// Gets the <see cref="Type"/> defining the model property represented by the current
         /// instance, or <c>null</c> if the current instance does not represent a property.
         /// </summary>
         public Type ContainerType { get; private set; }

src/Microsoft.AspNetCore.Mvc.Abstractions/ModelBinding/ModelMetadata.cs

@@ -419,7 +419,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
         /// <inheritdoc />
         public override int GetHashCode()
         {
-            // Normaly caching the hashcode would be dangerous, but Identity is deeply immutable so this is safe.
+            // Normally caching the hashcode would be dangerous, but Identity is deeply immutable so this is safe.
             if (_hashCode == null)
             {
                 _hashCode = Identity.GetHashCode();

src/Microsoft.AspNetCore.Mvc.ApiExplorer/DefaultApiDescriptionProvider.cs

@@ -610,7 +610,7 @@ namespace Microsoft.AspNetCore.Mvc.ApiExplorer
                 // For any property which is a leaf node, we don't want to keep traversing:
                 //
                 //  1)  Collections - while it's possible to have binder attributes on the inside of a collection,
-                //      it hardly seems useful, and would result in some very wierd binding.
+                //      it hardly seems useful, and would result in some very weird binding.
                 //
                 //  2)  Simple Types - These are generally part of the .net framework - primitives, or types which have a
                 //      type converter from string.

src/Microsoft.AspNetCore.Mvc.Core/AcceptedResult.cs

@@ -40,7 +40,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// provided.
         /// </summary>
         /// <param name="locationUri">The location at which the status of requested content can be monitored
-        /// It is an optional paramater and may be null</param>
+        /// It is an optional parameter and may be null</param>
         /// <param name="value">The value to format in the entity body.</param>
         public AcceptedResult(Uri locationUri, object value)
             : base(value)

src/Microsoft.AspNetCore.Mvc.Core/ApplicationModels/IParameterModelConvention.cs

@@ -4,7 +4,7 @@
 namespace Microsoft.AspNetCore.Mvc.ApplicationModels
 {
     /// <summary>
-    /// Allows customization of the of the <see cref="ControllerModel"/>.
+    /// Allows customization of the <see cref="ControllerModel"/>.
     /// </summary>
     /// <remarks>
     /// To use this interface, create an <see cref="System.Attribute"/> class which implements the interface and

src/Microsoft.AspNetCore.Mvc.Core/ApplicationModels/PropertyModel.cs

@@ -40,7 +40,7 @@ namespace Microsoft.AspNetCore.Mvc.ApplicationModels
         }
 
         /// <summary>
-        /// Creats a new instance of <see cref="PropertyModel"/> from a given <see cref="PropertyModel"/>.
+        /// Creates a new instance of <see cref="PropertyModel"/> from a given <see cref="PropertyModel"/>.
         /// </summary>
         /// <param name="other">The <see cref="PropertyModel"/> which needs to be copied.</param>
         public PropertyModel(PropertyModel other)

src/Microsoft.AspNetCore.Mvc.Core/ApplicationParts/AssemblyPart.cs

@@ -18,7 +18,7 @@ namespace Microsoft.AspNetCore.Mvc.ApplicationParts
         ICompilationReferencesProvider
     {
         /// <summary>
-        /// Initalizes a new <see cref="AssemblyPart"/> instance.
+        /// Initializes a new <see cref="AssemblyPart"/> instance.
         /// </summary>
         /// <param name="assembly"></param>
         public AssemblyPart(Assembly assembly)

src/Microsoft.AspNetCore.Mvc.Core/BindAttribute.cs

@@ -19,7 +19,7 @@ namespace Microsoft.AspNetCore.Mvc
         private Func<ModelMetadata, bool> _propertyFilter;
 
         /// <summary>
-        /// Creates a new instace of <see cref="BindAttribute"/>.
+        /// Creates a new instance of <see cref="BindAttribute"/>.
         /// </summary>
         /// <param name="include">Names of parameters to include in binding.</param>
         public BindAttribute(params string[] include)

src/Microsoft.AspNetCore.Mvc.Core/ControllerBase.cs

@@ -1842,7 +1842,7 @@ namespace Microsoft.AspNetCore.Mvc
             => new ChallengeResult(properties);
 
         /// <summary>
-        /// Creates a <see cref="ChallengeResult"/> with the specified specified authentication schemes and
+        /// Creates a <see cref="ChallengeResult"/> with the specified authentication schemes and
         /// <paramref name="properties" />.
         /// </summary>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the authentication
@@ -1903,7 +1903,7 @@ namespace Microsoft.AspNetCore.Mvc
 
         /// <summary>
         /// Creates a <see cref="ForbidResult"/> (<see cref="StatusCodes.Status403Forbidden"/> by default) with the 
-        /// specified specified authentication schemes and <paramref name="properties" />.
+        /// specified authentication schemes and <paramref name="properties" />.
         /// </summary>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the authentication
         /// challenge.</param>
@@ -1928,7 +1928,7 @@ namespace Microsoft.AspNetCore.Mvc
             => new SignInResult(authenticationScheme, principal);
 
         /// <summary>
-        /// Creates a <see cref="SignInResult"/> with the specified specified authentication scheme and
+        /// Creates a <see cref="SignInResult"/> with the specified authentication scheme and
         /// <paramref name="properties" />.
         /// </summary>
         /// <param name="principal">The <see cref="ClaimsPrincipal"/> containing the user claims.</param>
@@ -1952,7 +1952,7 @@ namespace Microsoft.AspNetCore.Mvc
             => new SignOutResult(authenticationSchemes);
 
         /// <summary>
-        /// Creates a <see cref="SignOutResult"/> with the specified specified authentication schemes and
+        /// Creates a <see cref="SignOutResult"/> with the specified authentication schemes and
         /// <paramref name="properties" />.
         /// </summary>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the sign-out operation.</param>

src/Microsoft.AspNetCore.Mvc.Core/Internal/AuthorizationApplicationModelProvider.cs

@@ -64,7 +64,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
         public static AuthorizeFilter GetFilter(IAuthorizationPolicyProvider policyProvider, IEnumerable<IAuthorizeData> authData)
         {
             // The default policy provider will make the same policy for given input, so make it only once.
-            // This will always execute syncronously.
+            // This will always execute synchronously.
             if (policyProvider.GetType() == typeof(DefaultAuthorizationPolicyProvider))
             {
                 var policy = AuthorizationPolicy.CombineAsync(policyProvider, authData).GetAwaiter().GetResult();

src/Microsoft.AspNetCore.Mvc.Core/Internal/DefaultApplicationModelProvider.cs

@@ -105,7 +105,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
                 throw new ArgumentNullException(nameof(typeInfo));
             }
 
-            // For attribute routes on a controller, we want want to support 'overriding' routes on a derived
+            // For attribute routes on a controller, we want to support 'overriding' routes on a derived
             // class. So we need to walk up the hierarchy looking for the first class to define routes.
             //
             // Then we want to 'filter' the set of attributes, so that only the effective routes apply.
@@ -291,7 +291,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
             //TODO: modify comment
             // Now we need to determine the action selection info (cross-section of routes and constraints)
             //
-            // For attribute routes on a action, we want want to support 'overriding' routes on a
+            // For attribute routes on a action, we want to support 'overriding' routes on a
             // virtual method, but allow 'overriding'. So we need to walk up the hierarchy looking
             // for the first definition to define routes.
             //
@@ -377,7 +377,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
                 return false;
             }
 
-            // Overriden methods from Object class, e.g. Equals(Object), GetHashCode(), etc., are not valid.
+            // Overridden methods from Object class, e.g. Equals(Object), GetHashCode(), etc., are not valid.
             if (methodInfo.GetBaseDefinition().DeclaringType == typeof(object))
             {
                 return false;

src/Microsoft.AspNetCore.Mvc.Core/Internal/DefaultModelBindingContext.cs

@@ -272,7 +272,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
             var scope = EnterNestedScope();
 
             // Only filter if the new BindingSource affects the value providers. Otherwise we want
-            // to preserve the currrent state.
+            // to preserve the current state.
             if (modelMetadata.BindingSource != null && !modelMetadata.BindingSource.IsGreedy)
             {
                 ValueProvider = FilterValueProvider(OriginalValueProvider, modelMetadata.BindingSource);

src/Microsoft.AspNetCore.Mvc.Core/Internal/ExplicitIndexCollectionValidationStrategy.cs

@@ -10,11 +10,11 @@ using Microsoft.AspNetCore.Mvc.ModelBinding.Validation;
 namespace Microsoft.AspNetCore.Mvc.Internal
 {
     /// <summary>
-    /// An implementation of <see cref="IValidationStrategy"/> for a collection bound using 'explict indexing'
+    /// An implementation of <see cref="IValidationStrategy"/> for a collection bound using 'explicit indexing'
     /// style keys.
     /// </summary>
     /// <remarks>
-    /// This implemenation handles cases like:
+    /// This implementation handles cases like:
     /// <example>
     ///     Model: IList&lt;Student&gt; 
     ///     Query String: ?students.index=Joey,Katherine&amp;students[Joey].Age=8&amp;students[Katherine].Age=9

src/Microsoft.AspNetCore.Mvc.Core/Internal/HttpTokenParsingRules.cs

@@ -152,7 +152,7 @@ namespace Microsoft.AspNetCore.Mvc.Formatters.Internal
                 return HttpParseResult.NotParsed;
             }
 
-            // Quoted-char has 2 characters. Check wheter there are 2 chars left ('\' + char)
+            // Quoted-char has 2 characters. Check whether there are 2 chars left ('\' + char)
             // If so, check whether the character is in the range 0-127. If not, it's an invalid value.
             if ((startIndex + 2 > input.Length) || (input[startIndex + 1] > 127))
             {

src/Microsoft.AspNetCore.Mvc.Core/Internal/MiddlewareFilter.cs

@@ -32,7 +32,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
             // Capture the current context into the feature. This will later be used in the end middleware to continue
             // the execution flow to later MVC layers.
             // Example:
-            // this filter -> user-middleware1 -> user-middleware2 -> the-end-middleware -> resouce filters or model binding
+            // this filter -> user-middleware1 -> user-middleware2 -> the-end-middleware -> resource filters or model binding
             var feature = new MiddlewareFilterFeature()
             {
                 ResourceExecutionDelegate = next,

src/Microsoft.AspNetCore.Mvc.Core/Internal/MiddlewareFilterBuilder.cs

@@ -58,7 +58,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
             // or model binding), add a middleware at the end of the user provided pipeline which make sure to continue
             // this flow.
             // Example:
-            // middleware filter -> user-middleware1 -> user-middleware2 -> end-middleware -> resouce filters or model binding
+            // middleware filter -> user-middleware1 -> user-middleware2 -> end-middleware -> resource filters or model binding
             nestedAppBuilder.Run(async (httpContext) =>
             {
                 var feature = httpContext.Features.Get<IMiddlewareFilterFeature>();

src/Microsoft.AspNetCore.Mvc.Core/Internal/ResourceInvoker.cs

@@ -526,7 +526,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
 
                         if (exceptionContext.Exception == null || exceptionContext.ExceptionHandled)
                         {
-                            // We don't need to do anthing to trigger a short circuit. If there's another
+                            // We don't need to do anything to trigger a short circuit. If there's another
                             // exception filter on the stack it will check the same set of conditions
                             // and then just skip itself.
                             _logger.ExceptionFilterShortCircuited(filter);
@@ -566,7 +566,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
 
                             if (exceptionContext.Exception == null || exceptionContext.ExceptionHandled)
                             {
-                                // We don't need to do anthing to trigger a short circuit. If there's another
+                                // We don't need to do anything to trigger a short circuit. If there's another
                                 // exception filter on the stack it will check the same set of conditions
                                 // and then just skip itself.
                                 _logger.ExceptionFilterShortCircuited(filter);

src/Microsoft.AspNetCore.Mvc.Core/Internal/ShortFormDictionaryValidationStrategy.cs

@@ -15,7 +15,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
     /// <typeparam name="TKey">The <see cref="Type"/> of the keys of the model dictionary.</typeparam>
     /// <typeparam name="TValue">The <see cref="Type"/> of the values of the model dictionary.</typeparam>
     /// <remarks>
-    /// This implemenation handles cases like:
+    /// This implementation handles cases like:
     /// <example>
     ///     Model: IDictionary&lt;string, Student&gt; 
     ///     Query String: ?students[Joey].Age=8&amp;students[Katherine].Age=9

src/Microsoft.AspNetCore.Mvc.Core/LocalRedirectResult.cs

@@ -43,7 +43,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// </summary>
         /// <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>
-        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the intial request's method.</param>
+        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the initial request's method.</param>
         public LocalRedirectResult(string localUrl, bool permanent, bool preserveMethod)
         {
             if (string.IsNullOrEmpty(localUrl))

src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/Metadata/DisplayMetadata.cs

@@ -42,7 +42,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding.Metadata
         public string DisplayFormatString { get; set; }
 
         /// <summary>
-        /// Gets or sets a delegate delegate which is used to get a value for the
+        /// Gets or sets a delegate which is used to get a value for the
         /// display name of the model. See <see cref="ModelMetadata.DisplayName"/>.
         /// </summary>
         public Func<string> DisplayName { get; set; }

src/Microsoft.AspNetCore.Mvc.Core/ModelBinding/ModelBinderFactory.cs

@@ -106,7 +106,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
                 return NoOpBinder.Instance;
             }
 
-            // A non-null token will usually be passed in at the the top level (ParameterDescriptor likely).
+            // A non-null token will usually be passed in at the top level (ParameterDescriptor likely).
             // This prevents us from treating a parameter the same as a collection-element - which could
             // happen looking at just model metadata.
             var key = new Key(providerContext.Metadata, token);
@@ -253,7 +253,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
                 }
 
                 // For non-root nodes we use the ModelMetadata as the cache token. This ensures that all non-root
-                // nodes with the same metadata will have the the same binder. This is OK because for an non-root
+                // nodes with the same metadata will have the same binder. This is OK because for an non-root
                 // node there's no opportunity to customize binding info like there is for a parameter.
                 var token = metadata;
 

src/Microsoft.AspNetCore.Mvc.Core/RedirectResult.cs

@@ -48,7 +48,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// </summary>
         /// <param name="url">The URL to redirect to.</param>
         /// <param name="permanent">Specifies whether the redirect should be permanent (301) or temporary (302).</param>
-        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the intial request method.</param>
+        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the initial request method.</param>
         public RedirectResult(string url, bool permanent, bool preserveMethod)
         {
             if (url == null)

src/Microsoft.AspNetCore.Mvc.Core/RedirectToActionResult.cs

@@ -73,7 +73,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <param name="controllerName">The name of the controller to use for generating the URL.</param>
         /// <param name="routeValues">The route data to use for generating the URL.</param>
         /// <param name="permanent">If set to true, makes the redirect permanent (301). Otherwise a temporary redirect is used (302).</param>
-        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the intial request method.</param>
+        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the initial request method.</param>
         public RedirectToActionResult(
             string actionName,
             string controllerName,
@@ -111,7 +111,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <param name="controllerName">The name of the controller to use for generating the URL.</param>
         /// <param name="routeValues">The route data to use for generating the URL.</param>
         /// <param name="permanent">If set to true, makes the redirect permanent (301). Otherwise a temporary redirect is used (302).</param>
-        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) and permanent redirect (308) preserve the intial request method.</param>
+        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) and permanent redirect (308) preserve the initial request method.</param>
         /// <param name="fragment">The fragment to add to the URL.</param>
         public RedirectToActionResult(
             string actionName,

src/Microsoft.AspNetCore.Mvc.Core/RedirectToPageResult.cs

@@ -84,7 +84,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <param name="pageHandler">The page handler to redirect to.</param>
         /// <param name="routeValues">The parameters for the page.</param>
         /// <param name="permanent">If set to true, makes the redirect permanent (301). Otherwise a temporary redirect is used (302).</param>
-        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the intial request method.</param>
+        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the initial request method.</param>
         public RedirectToPageResult(
             string pageName,
             string pageHandler,
@@ -140,7 +140,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <param name="pageHandler">The page handler to redirect to.</param>
         /// <param name="routeValues">The parameters for the page.</param>
         /// <param name="permanent">If set to true, makes the redirect permanent (301). Otherwise a temporary redirect is used (302).</param>
-        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the intial request method.</param>
+        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the initial request method.</param>
         /// <param name="fragment">The fragment to add to the URL.</param>
         public RedirectToPageResult(
             string pageName,
@@ -169,7 +169,7 @@ namespace Microsoft.AspNetCore.Mvc
         public string PageName { get; set; }
 
         /// <summary>
-        /// Gets or sets the the page handler to redirect to.
+        /// Gets or sets the page handler to redirect to.
         /// </summary>
         public string PageHandler { get; set; }
 

src/Microsoft.AspNetCore.Mvc.Core/RedirectToRouteResult.cs

@@ -61,7 +61,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <param name="routeName">The name of the route.</param>
         /// <param name="routeValues">The parameters for the route.</param>
         /// <param name="permanent">If set to true, makes the redirect permanent (301). Otherwise a temporary redirect is used (302).</param>
-        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the intial request method.</param>
+        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the initial request method.</param>
         public RedirectToRouteResult(
             string routeName,
             object routeValues,
@@ -110,7 +110,7 @@ namespace Microsoft.AspNetCore.Mvc
         /// <param name="routeName">The name of the route.</param>
         /// <param name="routeValues">The parameters for the route.</param>
         /// <param name="permanent">If set to true, makes the redirect permanent (301). Otherwise a temporary redirect is used (302).</param>
-        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the intial request method.</param>
+        /// <param name="preserveMethod">If set to true, make the temporary redirect (307) or permanent redirect (308) preserve the initial request method.</param>
         /// <param name="fragment">The fragment to add to the URL.</param>
         public RedirectToRouteResult(
             string routeName,

src/Microsoft.AspNetCore.Mvc.DataAnnotations/IAttributeAdapter.cs

@@ -6,7 +6,7 @@ using Microsoft.AspNetCore.Mvc.ModelBinding.Validation;
 namespace Microsoft.AspNetCore.Mvc.DataAnnotations
 {
     /// <summary>
-    /// Interface so that adapters provide their relevent values to error messages.
+    /// Interface so that adapters provide their relevant values to error messages.
     /// </summary>
     public interface IAttributeAdapter : IClientModelValidator
     {

src/Microsoft.AspNetCore.Mvc.DataAnnotations/Internal/DataAnnotationsMetadataProvider.cs

@@ -129,7 +129,7 @@ namespace Microsoft.AspNetCore.Mvc.DataAnnotations.Internal
             }
 
             // DisplayName
-            // DisplayAttribute has precendence over DisplayNameAttribute.
+            // DisplayAttribute has precedence over DisplayNameAttribute.
             if (displayAttribute?.GetName() != null)
             {
                 if (localizer != null &&

src/Microsoft.AspNetCore.Mvc.Formatters.Xml/WrapperProviderContext.cs

@@ -29,7 +29,7 @@ namespace Microsoft.AspNetCore.Mvc.Formatters.Xml
 
         /// <summary>
         /// The declared type which could be wrapped/un-wrapped by a different type 
-        /// during serialization or de-serializatoin.
+        /// during serialization or deserialization.
         /// </summary>
         public Type DeclaredType { get; }
 

src/Microsoft.AspNetCore.Mvc.Localization/IHtmlLocalizer.cs

@@ -8,7 +8,7 @@ using Microsoft.Extensions.Localization;
 namespace Microsoft.AspNetCore.Mvc.Localization
 {
     /// <summary>
-    /// Represents a type that that does HTML-aware localization of strings, by HTML encoding arguments that are
+    /// Represents a type that does HTML-aware localization of strings, by HTML encoding arguments that are
     /// formatted in the resource string.
     /// </summary>
     public interface IHtmlLocalizer

src/Microsoft.AspNetCore.Mvc.Razor/Compilation/CompiledViewDescriptor.cs

@@ -14,7 +14,7 @@ namespace Microsoft.AspNetCore.Mvc.Razor.Compilation
         public string RelativePath { get; set; }
 
         /// <summary>
-        /// Gets or sets the <see cref="RazorViewAttribute"/> decorating the sview.
+        /// Gets or sets the <see cref="RazorViewAttribute"/> decorating the view.
         /// </summary>
         public RazorViewAttribute ViewAttribute { get; set; }
 

src/Microsoft.AspNetCore.Mvc.Razor/IRazorPage.cs

@@ -14,7 +14,7 @@ namespace Microsoft.AspNetCore.Mvc.Razor
     public interface IRazorPage
     {
         /// <summary>
-        /// Gets or sets the view context of the renderign view.
+        /// Gets or sets the view context of the rendering view.
         /// </summary>
         ViewContext ViewContext { get; set; }
 

src/Microsoft.AspNetCore.Mvc.RazorPages/ApplicationModels/IPageApplicationModelConvention.cs

@@ -4,7 +4,7 @@
 namespace Microsoft.AspNetCore.Mvc.ApplicationModels
 {
     /// <summary>
-    /// Allows customization of the of the <see cref="PageApplicationModel"/>.
+    /// Allows customization of the <see cref="PageApplicationModel"/>.
     /// </summary>
     public interface IPageApplicationModelConvention : IPageConvention
     {

src/Microsoft.AspNetCore.Mvc.RazorPages/ApplicationModels/PageHandlerModel.cs

@@ -32,7 +32,7 @@ namespace Microsoft.AspNetCore.Mvc.ApplicationModels
         }
 
         /// <summary>
-        /// Creats a new instance of <see cref="PageHandlerModel"/> from a given <see cref="PageHandlerModel"/>.
+        /// Creates a new instance of <see cref="PageHandlerModel"/> from a given <see cref="PageHandlerModel"/>.
         /// </summary>
         /// <param name="other">The <see cref="PageHandlerModel"/> which needs to be copied.</param>
         public PageHandlerModel(PageHandlerModel other)

src/Microsoft.AspNetCore.Mvc.RazorPages/ApplicationModels/PagePropertyModel.cs

@@ -30,7 +30,7 @@ namespace Microsoft.AspNetCore.Mvc.ApplicationModels
         }
 
         /// <summary>
-        /// Creats a new instance of <see cref="PagePropertyModel"/> from a given <see cref="PagePropertyModel"/>.
+        /// Creates a new instance of <see cref="PagePropertyModel"/> from a given <see cref="PagePropertyModel"/>.
         /// </summary>
         /// <param name="other">The <see cref="PagePropertyModel"/> which needs to be copied.</param>
         public PagePropertyModel(PagePropertyModel other)

src/Microsoft.AspNetCore.Mvc.RazorPages/Internal/DefaultPageApplicationModelProvider.cs

@@ -249,7 +249,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorPages.Internal
                 return false;
             }
 
-            // Overriden methods from Object class, e.g. Equals(Object), GetHashCode(), etc., are not valid.
+            // Overridden methods from Object class, e.g. Equals(Object), GetHashCode(), etc., are not valid.
             if (methodInfo.GetBaseDefinition().DeclaringType == typeof(object))
             {
                 return false;

src/Microsoft.AspNetCore.Mvc.RazorPages/PageBase.cs

@@ -185,7 +185,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorPages
             => new ChallengeResult(properties);
 
         /// <summary>
-        /// Creates a <see cref="ChallengeResult"/> with the specified specified authentication schemes and
+        /// Creates a <see cref="ChallengeResult"/> with the specified authentication schemes and
         /// <paramref name="properties" />.
         /// </summary>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the authentication
@@ -296,7 +296,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorPages
 
         /// <summary>
         /// Creates a <see cref="ForbidResult"/> (<see cref="StatusCodes.Status403Forbidden"/> by default) with the 
-        /// specified specified authentication schemes and <paramref name="properties" />.
+        /// specified authentication schemes and <paramref name="properties" />.
         /// </summary>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the authentication
         /// challenge.</param>
@@ -1129,7 +1129,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorPages
             => new SignInResult(authenticationScheme, principal);
 
         /// <summary>
-        /// Creates a <see cref="SignInResult"/> with the specified specified authentication scheme and
+        /// Creates a <see cref="SignInResult"/> with the specified authentication scheme and
         /// <paramref name="properties" />.
         /// </summary>
         /// <param name="principal">The <see cref="ClaimsPrincipal"/> containing the user claims.</param>
@@ -1151,7 +1151,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorPages
             => new SignOutResult(authenticationSchemes);
 
         /// <summary>
-        /// Creates a <see cref="SignOutResult"/> with the specified specified authentication schemes and
+        /// Creates a <see cref="SignOutResult"/> with the specified authentication schemes and
         /// <paramref name="properties" />.
         /// </summary>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the sign-out operation.</param>

src/Microsoft.AspNetCore.Mvc.RazorPages/PageModel.cs

@@ -538,7 +538,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorPages
             => new ChallengeResult(properties);
 
         /// <summary>
-        /// Creates a <see cref="ChallengeResult"/> with the specified specified authentication schemes and
+        /// Creates a <see cref="ChallengeResult"/> with the specified authentication schemes and
         /// <paramref name="properties" />.
         /// </summary>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the authentication
@@ -649,7 +649,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorPages
 
         /// <summary>
         /// Creates a <see cref="ForbidResult"/> (<see cref="StatusCodes.Status403Forbidden"/> by default) with the 
-        /// specified specified authentication schemes and <paramref name="properties" />.
+        /// specified authentication schemes and <paramref name="properties" />.
         /// </summary>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the authentication
         /// challenge.</param>
@@ -1502,7 +1502,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorPages
             => new SignInResult(authenticationScheme, principal);
 
         /// <summary>
-        /// Creates a <see cref="SignInResult"/> with the specified specified authentication scheme and
+        /// Creates a <see cref="SignInResult"/> with the specified authentication scheme and
         /// <paramref name="properties" />.
         /// </summary>
         /// <param name="principal">The <see cref="ClaimsPrincipal"/> containing the user claims.</param>
@@ -1524,7 +1524,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorPages
             => new SignOutResult(authenticationSchemes);
 
         /// <summary>
-        /// Creates a <see cref="SignOutResult"/> with the specified specified authentication schemes and
+        /// Creates a <see cref="SignOutResult"/> with the specified authentication schemes and
         /// <paramref name="properties" />.
         /// </summary>
         /// <param name="properties"><see cref="AuthenticationProperties"/> used to perform the sign-out operation.</param>

src/Microsoft.AspNetCore.Mvc.Testing/MvcWebApplicationBuilder.cs

@@ -65,7 +65,7 @@ namespace Microsoft.AspNetCore.Mvc.Testing
                     "'<PreserveCompilationContext>true</PreserveCompilationContext>'." +
                     $"For functional tests to work they need to either run from the build output folder or the {Path.GetFileName(depsFile.FullName)} " +
                     $"file from your application's output directory must be copied" +
-                    "to the folder where the tests are running on. A common cause for this error is having shadow copying enabled when the" +
+                    "to the folder where the tests are running on. A common cause for this error is having shadow copying enabled when the " +
                     "tests run.");
             }
 

src/Microsoft.AspNetCore.Mvc.Testing/WebApplicationTestFixture.cs

@@ -67,7 +67,7 @@ namespace Microsoft.AspNetCore.Mvc.Testing
         /// The startup code defined in <typeparamref name = "TStartup" /> will be executed to configure the application.
         /// </para>
         /// <para>
-        /// This constructor will infer the application root directive by searching for a solution file tht matches the pattern
+        /// This constructor will infer the application root directive by searching for a solution file that matches the pattern
         /// <paramref name="solutionSearchPattern"/> and then appending the path <paramref name="solutionRelativePath"/>
         /// to the solution directory.The application root directory will be used to discover views and content files.
         /// </para>

src/Microsoft.AspNetCore.Mvc.Testing/build/netstandard2.0/Microsoft.AspNetCore.Mvc.Testing.targets

@@ -9,7 +9,7 @@
       The functional tests act as the host application for all test websites. Since the CLI copies all reference
       assembly dependencies in websites to their corresponding bin/{config}/refs folder we need to re-calculate
       reference assemblies for this project so there's a corresponding refs folder in our output. Without it
-      our websites deps files will fail to find their assembly referenes.
+      our websites deps files will fail to find their assembly references.
     -->
 
     <PreserveCompilationContext>true</PreserveCompilationContext>

src/Microsoft.AspNetCore.Mvc.ViewFeatures/TempDataAttribute.cs

@@ -8,7 +8,7 @@ namespace Microsoft.AspNetCore.Mvc
     /// <summary>
     /// Properties decorated with <see cref="TempDataAttribute"/> will have their values stored in
     /// and loaded from the <see cref="ViewFeatures.TempDataDictionary"/>. <see cref="TempDataAttribute"/>
-    /// is supported on properties of Controllers, Razor Pages, and Razor Page Page Models.
+    /// is supported on properties of Controllers, Razor Pages, and Razor Page Models.
     /// </summary>
     [AttributeUsage(AttributeTargets.Property, Inherited = true, AllowMultiple = false)]
     public sealed class TempDataAttribute : Attribute

src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewComponents/ViewComponentDescriptor.cs

@@ -56,7 +56,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewComponents
         /// <remarks>
         /// <para>
         /// The full name is defaulted to the full namespace of the view component class, prepended to
-        /// the the class name with a '.' character as the separator. If the view component class uses
+        /// the class name with a '.' character as the separator. If the view component class uses
         /// <code>ViewComponent</code> as a suffix, the suffix will be omitted from the <see cref="FullName"/>.
         /// </para>
         /// <example>

src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/IHtmlGenerator.cs

@@ -29,7 +29,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// <param name="controllerName">The name of the controller.</param>
         /// <param name="protocol">The protocol (scheme) 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="fragment">The fragment for the generated 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
@@ -63,7 +63,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// <param name="pageHandler">The page handler.</param>
         /// <param name="protocol">The protocol (scheme) 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="fragment">The fragment for the generated 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
@@ -254,7 +254,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// <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="fragment">The fragment for the genrated link.</param>
+        /// <param name="fragment">The fragment for the generated 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

src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/SaveTempDataAttribute.cs

@@ -17,7 +17,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         public SaveTempDataAttribute()
         {
             // Since SaveTempDataFilter registers for a response's OnStarting callback, we want this filter to run 
-            // as early as possible to get the oppurtunity to register the call back before any other result filter 
+            // as early as possible to get the opportunity to register the call back before any other result filter 
             // starts writing to the response stream.
             Order = int.MinValue + 100;
         }

src/Microsoft.AspNetCore.Mvc.ViewFeatures/ViewFeatures/ViewDataInfo.cs

@@ -32,7 +32,7 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures
         /// passing parameter <paramref name="container"/> to lazily evaluate the value.
         /// </summary>
         /// <param name="container">The <see cref="object"/> that <see cref="Value"/> will be evaluated from.</param>
-        /// <param name="propertyInfo">The <see cref="PropertyInfo"/> that will be used to evalute <see cref="Value"/>.</param>
+        /// <param name="propertyInfo">The <see cref="PropertyInfo"/> that will be used to evaluate <see cref="Value"/>.</param>
         public ViewDataInfo(object container, PropertyInfo propertyInfo)
             : this(container, propertyInfo, _propertyInfoResolver)
         {

test/Microsoft.AspNetCore.Mvc.Core.Test/ContentResultTest.cs

@@ -261,7 +261,7 @@ namespace Microsoft.AspNetCore.Mvc
         private static IServiceCollection CreateServices(params ViewComponentDescriptor[] descriptors)
         {
             // An array pool could return a buffer which is greater or equal to the size of the default character
-            // chunk size. Since the tests here depend on a specifc character buffer size to test boundary conditions,
+            // chunk size. Since the tests here depend on a specific character buffer size to test boundary conditions,
             // make sure to only return a buffer of that size.
             var charArrayPool = new Mock<ArrayPool<char>>();
             charArrayPool

test/Microsoft.AspNetCore.Mvc.Core.Test/Internal/DefaultAssemblyPartDiscoveryProviderTest.cs

@@ -227,7 +227,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
         }
 
         // This test verifies DefaultAssemblyPartDiscoveryProvider.ReferenceAssemblies reflects the actual loadable assemblies
-        // of the libraries that Microsoft.AspNetCore.Mvc dependes on.
+        // of the libraries that Microsoft.AspNetCore.Mvc depends on.
         // If we add or remove dependencies, this test should be changed together.
         [Fact]
         public void ReferenceAssemblies_ReturnsLoadableReferenceAssemblies()

test/Microsoft.AspNetCore.Mvc.Core.Test/Internal/ResponseContentTypeHelperTest.cs

@@ -81,7 +81,7 @@ namespace Microsoft.AspNetCore.Mvc.Internal
                         "text/bar; p1=p1-value; charset=us-ascii"
                     },
 
-                    // Content type set on action result takes precedence over the conten type set on http response
+                    // Content type set on action result takes precedence over the content type set on http response
                     {
                         MediaTypeHeaderValue.Parse("text/foo; charset=us-ascii"),
                         "text/bar",

test/Microsoft.AspNetCore.Mvc.Core.Test/ModelBinding/Internal/ValidationStackTest.cs

@@ -57,7 +57,7 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding.Internal
             PreLoad(preloadCount, validationStack);
 
             // Act & Assert
-            // Poping null when it's not there must not throw
+            // Popping null when it's not there must not throw
             validationStack.Pop(null);
         }
 

test/Microsoft.AspNetCore.Mvc.Core.Test/ModelBinding/ModelBinderFactoryTest.cs

@@ -530,8 +530,8 @@ namespace Microsoft.AspNetCore.Mvc.ModelBinding
             Assert.Equal(1, widgetIdProvider.SuccessCount);
         }
 
-        // This is a really wierd case, but I wanted to make sure it's covered so it doesn't
-        // blow up in wierd ways.
+        // This is a really weird case, but I wanted to make sure it's covered so it doesn't
+        // blow up in weird ways.
         //
         // If a binder provider tries to recursively create itself, but then returns null, we've
         // already returned and possibly cached the PlaceholderBinder instance, we want to make sure that

test/Microsoft.AspNetCore.Mvc.Core.Test/Routing/UrlHelperTest.cs

@@ -791,7 +791,7 @@ namespace Microsoft.AspNetCore.Mvc.Routing
         [Fact]
         public void Action_RouteValueInvalidation_AffectsOtherRouteValues()
         {
-            // Arrage
+            // Arrange
             var services = CreateServices();
             var routeBuilder = CreateRouteBuilder(services);
 
@@ -835,7 +835,7 @@ namespace Microsoft.AspNetCore.Mvc.Routing
         [Fact]
         public void Action_RouteValueInvalidation_DoesNotAffectActionAndController_ActionPassedInRouteValues()
         {
-            // Arrage
+            // Arrange
             var services = CreateServices();
             var routeBuilder = CreateRouteBuilder(services);
 
@@ -898,7 +898,7 @@ namespace Microsoft.AspNetCore.Mvc.Routing
             string virtualPath,
             string expected)
         {
-            // Arrage
+            // Arrange
             var router = Mock.Of<IRouter>();
             var pathData = new VirtualPathData(router, virtualPath)
             {
@@ -921,7 +921,7 @@ namespace Microsoft.AspNetCore.Mvc.Routing
             string virtualPath,
             string expected)
         {
-            // Arrage
+            // Arrange
             var fragmentValue = "fragment-value";
             expected += $"#{fragmentValue}";
             var router = Mock.Of<IRouter>();
@@ -954,7 +954,7 @@ namespace Microsoft.AspNetCore.Mvc.Routing
             string fragment,
             string expected)
         {
-            // Arrage
+            // Arrange
             var router = Mock.Of<IRouter>();
             var pathData = new VirtualPathData(router, virtualPath)
             {

test/Microsoft.AspNetCore.Mvc.FunctionalTests/ContentNegotiationTest.cs

@@ -273,7 +273,7 @@ END:VCARD
         [Fact]
         public async Task ProducesAttribute_OnDerivedAction_OverridesTheValueOnBaseAction()
         {
-            // Arange
+            // Arrange
             var expectedContentType = MediaTypeHeaderValue.Parse(
                 "application/custom_NoProducesContentOnClassController_Action;charset=utf-8");
             var expectedBody = "NoProducesContentOnClassController";

test/Microsoft.AspNetCore.Mvc.FunctionalTests/HtmlGenerationTest.cs

@@ -523,7 +523,7 @@ Products: Music Systems, Televisions (3)";
             Assert.Equal(expected, response, ignoreLineEndingDifferences: true);
         }
 
-        // We want to make sure that for 'wierd' model expressions involving:
+        // We want to make sure that for 'weird' model expressions involving:
         // - fields
         // - statics
         // - private
@@ -531,7 +531,7 @@ Products: Music Systems, Televisions (3)";
         // These tests verify that we don't throw, and can evaluate the expression to get the model
         // value. One quirk of behavior for these cases is that we can't return a correct model metadata
         // instance (this is true for anything other than a public instance property). We're not overly
-        // concerned with that, and so the accuracy of the model metadata is is not verified by the test.
+        // concerned with that, and so the accuracy of the model metadata is not verified by the test.
         [Theory]
         [InlineData("GetWeirdWithHtmlHelpers")]
         [InlineData("GetWeirdWithTagHelpers")]

test/Microsoft.AspNetCore.Mvc.FunctionalTests/InputObjectValidationTests.cs

@@ -68,7 +68,7 @@ namespace Microsoft.AspNetCore.Mvc.FunctionalTests
 
             // Assert
             Assert.Equal(HttpStatusCode.OK, response.StatusCode);
-            Assert.Equal("User has been registerd : " + sampleName,
+            Assert.Equal("User has been registered : " + sampleName,
                 await response.Content.ReadAsStringAsync());
         }
 

test/Microsoft.AspNetCore.Mvc.IntegrationTests/ActionParametersIntegrationTest.cs

@@ -597,7 +597,7 @@ namespace Microsoft.AspNetCore.Mvc.IntegrationTests
         }
 
         // By default the ComplexTypeModelBinder fails to construct models for types with no parameterless constructor,
-        // but a developer could change this behavior by overridng CreateModel
+        // but a developer could change this behavior by overriding CreateModel
         private class CustomComplexTypeModelBinder : ComplexTypeModelBinder
         {
             public CustomComplexTypeModelBinder(IDictionary<ModelMetadata, IModelBinder> propertyBinders)

test/Microsoft.AspNetCore.Mvc.RazorPages.Test/Internal/DefaultPageApplicationModelProviderTest.cs

@@ -332,7 +332,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorPages.Internal
                 });
         }
 
-        // We want to test the the 'empty' page has no bound properties, and no handler methods.
+        // We want to test the 'empty' page has no bound properties, and no handler methods.
         [Fact]
         public void OnProvidersExecuting_EmptyPage()
         {
@@ -353,7 +353,7 @@ namespace Microsoft.AspNetCore.Mvc.RazorPages.Internal
             Assert.Same(typeof(EmptyPage).GetTypeInfo(), pageModel.PageType);
         }
 
-        // We want to test the the 'empty' page and pagemodel has no bound properties, and no handler methods.
+        // We want to test the 'empty' page and pagemodel has no bound properties, and no handler methods.
         [Fact]
         public void OnProvidersExecuting_EmptyPageModel()
         {

test/Microsoft.AspNetCore.Mvc.Test/MvcServiceCollectionExtensionsTest.cs

@@ -57,7 +57,7 @@ namespace Microsoft.AspNetCore.Mvc
             var services = new ServiceCollection();
             services.AddSingleton<IHostingEnvironment>(GetHostingEnvironment());
 
-            // Register a mock implementation of each service, AddMvcServices should add another implemenetation.
+            // Register a mock implementation of each service, AddMvcServices should add another implementation.
             foreach (var serviceType in MutliRegistrationServiceTypes)
             {
                 var mockType = typeof(Mock<>).MakeGenericType(serviceType.Key);

test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/ControllerUnitTestabilityTests.cs

@@ -15,7 +15,7 @@ using Xunit;
 
 namespace Microsoft.AspNetCore.Mvc
 {
-    // Tests Controler for the unit testability with which users can simply instantiate contollers for unit tests
+    // Tests Controller for the unit testability with which users can simply instantiate controllers for unit tests
     public class ControllerUnitTestabilityTests
     {
         [Theory]

test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Internal/ExpressionHelperTest.cs

@@ -129,12 +129,12 @@ namespace Microsoft.AspNetCore.Mvc.ViewFeatures.Internal
                     },
                     // Constants are not supported.
                     {
-                        // Namespace never appears in expresison name. "Model" there doesn't matter.
+                        // Namespace never appears in expression name. "Model" there doesn't matter.
                         (Expression<Func<TestModel, int>>)(m => Microsoft.AspNetCore.Mvc.ViewFeatures.Model.Constants.WoodstockYear),
                         string.Empty
                     },
                     {
-                        // Class name never appears in expresion name. "Model" there doesn't matter.
+                        // Class name never appears in expression name. "Model" there doesn't matter.
                         (Expression<Func<TestModel, int>>)(m => Model.Constants.WoodstockYear),
                         string.Empty
                     },

test/Microsoft.AspNetCore.Mvc.ViewFeatures.Test/Rendering/HtmlHelperFormTest.cs

@@ -589,7 +589,7 @@ namespace Microsoft.AspNetCore.Mvc.Rendering
             // Act & Assert
             using (var form = htmlHelper.BeginForm(FormMethod.Post, antiforgery: false, htmlAttributes: null))
             {
-                // This call will ouput a token.
+                // This call will output a token.
                 Assert.Equal("antiforgery", Assert.IsType<TagBuilder>(htmlHelper.AntiForgeryToken()).TagName);
             }
 

test/Microsoft.AspNetCore.Mvc.WebApiCompatShimTest/TestUtils/TestData.cs

@@ -273,7 +273,7 @@ namespace Microsoft.TestCommon
         /// <param name="variation">The test data variation with which to create the related <see cref="Type"/>.</param>
         /// <returns>The related <see cref="Type"/> for the <see cref="TestData.Type"/> as given by the test data variation.</returns>
         /// <example>
-        /// For example, if the given <see cref="TestData"/> was created for <see cref="string"/> test data and the varation parameter
+        /// For example, if the given <see cref="TestData"/> was created for <see cref="string"/> test data and the variation parameter
         /// was <see cref="TestDataVariations.AsList"/> then the returned type would be <see cref="List<string>"/>.
         /// </example>
         public Type GetAsTypeOrNull(TestDataVariations variation)

test/WebSites/BasicWebSite/_gruntfile.js

@@ -15,6 +15,6 @@ module.exports = function (grunt) {
     grunt.registerTask("default", ["bower:install"]);
 
     // The following line loads the grunt plugins.
-    // This line needs to be at the end of this this file.
+    // This line needs to be at the end of this file.
     grunt.loadNpmTasks("grunt-bower-task");
 };

test/WebSites/FormatterWebSite/Controllers/ValidationController.cs

@@ -19,7 +19,7 @@ namespace FormatterWebSite
                     ModelState["Designation"].Errors[0].ErrorMessage);
             }
 
-            return Content("User has been registerd : " + user.Name);
+            return Content("User has been registered : " + user.Name);
         }
 
         [HttpPost]

test/WebSites/WebApiCompatShimWebSite/Controllers/ActionSelection/Legacy/EnumParameterOverloadsController.cs

@@ -7,7 +7,7 @@ using System.Web.Http;
 
 namespace WebApiCompatShimWebSite
 {
-    // This was ported from the WebAPI 5.2 codebase. Kept the same intentionally for compatability.
+    // This was ported from the WebAPI 5.2 codebase. Kept the same intentionally for compatibility.
     [ActionSelectionFilter]
     public class EnumParameterOverloadsController : ApiController
     {

test/WebSites/WebApiCompatShimWebSite/Controllers/ActionSelection/Legacy/ParameterAttributeController.cs

@@ -7,7 +7,7 @@ using Microsoft.AspNetCore.Mvc;
 
 namespace WebApiCompatShimWebSite
 {
-    // This was ported from the WebAPI 5.2 codebase. Kept the same intentionally for compatability.
+    // This was ported from the WebAPI 5.2 codebase. Kept the same intentionally for compatibility.
     [ActionSelectionFilter]
     public class ParameterAttributeController : ApiController
     {

test/WebSites/WebApiCompatShimWebSite/Controllers/ActionSelection/Legacy/TestController.cs

@@ -7,7 +7,7 @@ using Microsoft.AspNetCore.Mvc;
 
 namespace WebApiCompatShimWebSite
 {
-    // This was ported from the WebAPI 5.2 codebase. Kept the same intentionally for compatability.
+    // This was ported from the WebAPI 5.2 codebase. Kept the same intentionally for compatibility.
     [ActionSelectionFilter]
     public class TestController : ApiController
     {