PR feedback

Author: JamesNK

src/Http/Authentication.Abstractions/src/IAuthenticationService.cs

@@ -40,7 +40,7 @@ public interface IAuthenticationService
     Task ForbidAsync(HttpContext context, string? scheme, AuthenticationProperties? properties);
 
     /// <summary>
-    /// Sign in a principal in for the specified authentication scheme.
+    /// Sign a principal in for the specified authentication scheme.
     /// </summary>
     /// <param name="context">The <see cref="HttpContext"/>.</param>
     /// <param name="scheme">The name of the authentication scheme.</param>

src/Http/Authentication.Core/src/AuthenticationService.cs

@@ -145,7 +145,7 @@ public virtual async Task ForbidAsync(HttpContext context, string? scheme, Authe
     }
 
     /// <summary>
-    /// Sign in a principal in for the specified authentication scheme.
+    /// Sign a principal in for the specified authentication scheme.
     /// </summary>
     /// <param name="context">The <see cref="HttpContext"/>.</param>
     /// <param name="scheme">The name of the authentication scheme.</param>

src/Identity/Extensions.Core/src/UserManager.cs

@@ -1411,7 +1411,7 @@ public virtual async Task<IdentityResult> RemoveFromRolesAsync(TUser user, IEnum
     {
         try
         {
-            var result = await ResolveFromtRolesCoreAsync(user, roles).ConfigureAwait(false);
+            var result = await RemoveFromRolesCoreAsync(user, roles).ConfigureAwait(false);
             _metrics?.UpdateUser(typeof(TUser).FullName!, result, UserUpdateType.RemoveFromRoles);
             return result;
         }
@@ -1422,7 +1422,7 @@ public virtual async Task<IdentityResult> RemoveFromRolesAsync(TUser user, IEnum
         }
     }
 
-    private async Task<IdentityResult> ResolveFromtRolesCoreAsync(TUser user, IEnumerable<string> roles)
+    private async Task<IdentityResult> RemoveFromRolesCoreAsync(TUser user, IEnumerable<string> roles)
     {
         ThrowIfDisposed();
         var userRoleStore = GetUserRoleStore();