From f9513d7eff4ce5ad3973931dafe00c49a9da0cc4 Mon Sep 17 00:00:00 2001
From: "Yuzhi Chen (Wicresoft)" <v-yuzhichen@microsoft.com>
Date: Mon, 13 Jun 2022 18:35:23 +0800
Subject: [PATCH 1/3] Fix Az.Resources XML Comments

---
 .../Models.Authorization/AuthorizationClient.cs       | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/Resources/Resources/Models.Authorization/AuthorizationClient.cs b/src/Resources/Resources/Models.Authorization/AuthorizationClient.cs
index 0cb12787d690..9ddd87055755 100644
--- a/src/Resources/Resources/Models.Authorization/AuthorizationClient.cs
+++ b/src/Resources/Resources/Models.Authorization/AuthorizationClient.cs
@@ -61,6 +61,7 @@ public PSRoleDefinition GetRoleDefinition(string roleId)
         /// Gets a single role definition by the role Id guid.
         /// </summary>
         /// <param name="roleId">RoleId guid</param>
+        /// <param name="scope"></param>
         public PSRoleDefinition GetRoleDefinition(Guid roleId, string scope)
         {
             return AuthorizationManagementClient.RoleDefinitions.Get(scope, roleId.ToString()).ToPSRoleDefinition();
@@ -72,6 +73,9 @@ public PSRoleDefinition GetRoleDefinition(Guid roleId, string scope)
         /// Otherwise  will fetch Roledefinitions with provided name
         /// </summary>
         /// <param name="name">The role name</param>
+        /// <param name="scope"></param>
+        /// <param name="first"></param>
+        /// <param name="skip"></param>
         /// <returns>The matched role Definitions</returns>
         public IEnumerable<PSRoleDefinition> FilterRoleDefinitions(string name, string scope, ulong first = ulong.MaxValue, ulong skip = 0)
         {
@@ -125,6 +129,7 @@ public IEnumerable<PSRoleDefinition> FilterRoleDefinitionsByCustom(string scope,
         /// Creates new role assignment.
         /// </summary>
         /// <param name="parameters">The create parameters</param>
+        /// <param name="roleAssignmentId"></param>
         /// <returns>The created role assignment object</returns>
         public PSRoleAssignment CreateRoleAssignment(FilterRoleAssignmentsOptions parameters, Guid roleAssignmentId = default(Guid))
         {
@@ -173,6 +178,8 @@ public IEnumerable<PSRoleDefinition> FilterRoleDefinitionsByCustom(string scope,
         /// </summary>
         /// <param name="options">The filtering options</param>
         /// <param name="currentSubscription">The current subscription</param>
+        /// <param name="first"></param>
+        /// <param name="skip"></param>
         /// <returns>The filtered role assignments</returns>
         public List<PSRoleAssignment> FilterRoleAssignments(FilterRoleAssignmentsOptions options, string currentSubscription, ulong first = ulong.MaxValue, ulong skip = 0)
         {
@@ -382,7 +389,7 @@ public PSRoleDefinition GetSingleRoleDefinitionByName(string name, string scope)
         /// Deletes a role definition based on the id.
         /// </summary>
         /// <param name="roleDefinitionId">The role definition id to delete</param>
-        /// <param name="subscriptionId">Current subscription id</param>
+        /// <param name="scope"></param>
         /// <returns>The deleted role definition.</returns>
         public PSRoleDefinition RemoveRoleDefinition(Guid roleDefinitionId, string scope)
         {
@@ -401,6 +408,7 @@ public PSRoleDefinition RemoveRoleDefinition(Guid roleDefinitionId, string scope
         /// Deletes a role definition based on the name.
         /// </summary>
         /// <param name="roleDefinitionName">The role definition name.</param>
+        /// <param name="scope"></param>
         /// <returns>The deleted role definition.</returns>
         public PSRoleDefinition RemoveRoleDefinition(string roleDefinitionName, string scope)
         {
@@ -467,6 +475,7 @@ public PSRoleDefinition UpdateRoleDefinition(PSRoleDefinition roleDefinition)
         /// Creates a new role definition.
         /// </summary>
         /// <param name="roleDefinition">The role definition to create.</param>
+        /// <param name="roleDefinitionId">The role definition id to delete</param>
         /// <returns>The created role definition.</returns>
         public PSRoleDefinition CreateRoleDefinition(PSRoleDefinition roleDefinition, Guid roleDefinitionId = default(Guid))
         {

From f218b8595b03c60c3100243aa3ddd9b60a1845c2 Mon Sep 17 00:00:00 2001
From: "Yuzhi Chen (Wicresoft)" <v-yuzhichen@microsoft.com>
Date: Fri, 17 Jun 2022 12:22:33 +0800
Subject: [PATCH 2/3] Fix Az.Resources XML COmments

---
 .../ResourceManager/Components/ApiVersionHelper.cs          | 3 +++
 .../ResourceManager/Components/HttpClientHelper.cs          | 1 +
 .../ResourceManager/Components/HttpClientHelperFactory.cs   | 2 ++
 .../ResourceManager/Components/QueryFilterBuilder.cs        | 2 ++
 .../ResourceManager/Components/ResourceIdUtility.cs         | 5 ++---
 .../Components/ResourceManagerClientHelper.cs               | 1 +
 .../Components/TemplateSpecPackagingEngine.cs               | 2 ++
 .../Extensions/ErrorResponseMessageExceptionExtensions.cs   | 1 -
 .../ResourceManager/Extensions/PsObjectExtensions.cs        | 6 +++---
 src/Resources/ResourceManager/Handlers/CmdletInfoHandler.cs | 2 +-
 .../Features/UnregisterAzureProviderPreviewFeatureCmdlet.cs | 6 +++---
 .../ManagedApplications/ManagedApplicationCmdletBase.cs     | 1 +
 .../Implementation/Policy/PolicyCmdletBase.cs               | 2 --
 .../ResourceManager/SdkClient/ResourceManagerSdkClient.cs   | 2 +-
 .../ResourceManager/SdkClient/TemplateSpecsSdkClient.cs     | 2 +-
 15 files changed, 23 insertions(+), 15 deletions(-)

diff --git a/src/Resources/ResourceManager/Components/ApiVersionHelper.cs b/src/Resources/ResourceManager/Components/ApiVersionHelper.cs
index 4e76b2b4f20c..671959597508 100644
--- a/src/Resources/ResourceManager/Components/ApiVersionHelper.cs
+++ b/src/Resources/ResourceManager/Components/ApiVersionHelper.cs
@@ -46,6 +46,7 @@ internal static class ApiVersionHelper
         /// <param name="resourceId">The resource Id.</param>
         /// <param name="cancellationToken">The cancellation token</param>
         /// <param name="pre">When specified, indicates if pre-release API versions should be considered.</param>
+        /// <param name="cmdletHeaderValues"></param>
         internal static Task<string> DetermineApiVersion(IAzureContext context, string resourceId, CancellationToken cancellationToken, bool? pre = null, Dictionary<string, string> cmdletHeaderValues = null)
         {
             var providerNamespace = ResourceIdUtility.GetExtensionProviderNamespace(resourceId)
@@ -65,6 +66,7 @@ internal static Task<string> DetermineApiVersion(IAzureContext context, string r
         /// <param name="resourceType">The resource type.</param>
         /// <param name="cancellationToken">The cancellation token</param>
         /// <param name="pre">When specified, indicates if pre-release API versions should be considered.</param>
+        /// <param name="cmdletHeaderValues"></param>
         internal static Task<string> DetermineApiVersion(IAzureContext context, string providerNamespace, string resourceType, CancellationToken cancellationToken, bool? pre = null, Dictionary<string, string> cmdletHeaderValues = null)
         {
             var cacheKey = ApiVersionCache.GetCacheKey(context.Environment.Name, providerNamespace: providerNamespace, resourceType: resourceType);
@@ -106,6 +108,7 @@ internal static Task<string> DetermineApiVersion(IAzureContext context, string p
         /// <param name="providerNamespace">The provider namespace.</param>
         /// <param name="resourceType">The resource type.</param>
         /// <param name="cancellationToken">The cancellation token.</param>
+        /// <param name="cmdletHeaderValues"></param>
         private static string[] GetApiVersionsForResourceType(IAzureContext context, string providerNamespace, string resourceType, CancellationToken cancellationToken, Dictionary<string, string> cmdletHeaderValues = null)
         {
             var resourceManagerClient = ResourceManagerClientHelper.GetResourceManagerClient(context, cmdletHeaderValues);
diff --git a/src/Resources/ResourceManager/Components/HttpClientHelper.cs b/src/Resources/ResourceManager/Components/HttpClientHelper.cs
index 8355c2833a8b..67534a5d14a2 100644
--- a/src/Resources/ResourceManager/Components/HttpClientHelper.cs
+++ b/src/Resources/ResourceManager/Components/HttpClientHelper.cs
@@ -47,6 +47,7 @@ public abstract class HttpClientHelper
         /// </summary>
         /// <param name="credentials">The service client credentials.</param>
         /// <param name="headerValues">The header values.</param>
+        /// <param name="cmdletHeaderValues"></param>
         protected HttpClientHelper(ServiceClientCredentials credentials, IEnumerable<ProductInfoHeaderValue> headerValues, Dictionary<string, string> cmdletHeaderValues)
         {
             this.credentials = credentials;
diff --git a/src/Resources/ResourceManager/Components/HttpClientHelperFactory.cs b/src/Resources/ResourceManager/Components/HttpClientHelperFactory.cs
index 1a018e571395..584fd17c4667 100644
--- a/src/Resources/ResourceManager/Components/HttpClientHelperFactory.cs
+++ b/src/Resources/ResourceManager/Components/HttpClientHelperFactory.cs
@@ -48,6 +48,7 @@ protected HttpClientHelperFactory()
         /// </summary>
         /// <param name="credentials">The credentials.</param>
         /// <param name="headerValues">The headers.</param>
+        /// <param name="cmdletHeaderValues"></param>
         public virtual HttpClientHelper CreateHttpClientHelper(ServiceClientCredentials credentials, IEnumerable<ProductInfoHeaderValue> headerValues, Dictionary<string, string> cmdletHeaderValues)
         {
             return new HttpClientHelperImpl(credentials: credentials, headerValues: headerValues, cmdletHeaderValues: cmdletHeaderValues);
@@ -63,6 +64,7 @@ private class HttpClientHelperImpl : HttpClientHelper
             /// </summary>
             /// <param name="credentials">The credentials.</param>
             /// <param name="headerValues">The headers.</param>
+            /// <param name="cmdletHeaderValues"></param>
             public HttpClientHelperImpl(ServiceClientCredentials credentials, IEnumerable<ProductInfoHeaderValue> headerValues, Dictionary<string, string> cmdletHeaderValues)
                 : base(credentials: credentials, headerValues: headerValues, cmdletHeaderValues: cmdletHeaderValues)
             {
diff --git a/src/Resources/ResourceManager/Components/QueryFilterBuilder.cs b/src/Resources/ResourceManager/Components/QueryFilterBuilder.cs
index d366063c0626..043957700fbb 100644
--- a/src/Resources/ResourceManager/Components/QueryFilterBuilder.cs
+++ b/src/Resources/ResourceManager/Components/QueryFilterBuilder.cs
@@ -34,6 +34,8 @@ public static class QueryFilterBuilder
         /// <param name="tagName">The tag name.</param>
         /// <param name="tagValue">The tag value.</param>
         /// <param name="filter">The filter.</param>
+        /// <param name="nameContains"></param>
+        /// <param name="resourceGroupNameContains"></param>
         public static string CreateFilter(
             string subscriptionId,
             string resourceGroup,
diff --git a/src/Resources/ResourceManager/Components/ResourceIdUtility.cs b/src/Resources/ResourceManager/Components/ResourceIdUtility.cs
index c54653ad06bf..db1f43ccf738 100644
--- a/src/Resources/ResourceManager/Components/ResourceIdUtility.cs
+++ b/src/Resources/ResourceManager/Components/ResourceIdUtility.cs
@@ -42,7 +42,6 @@ public static class ResourceIdUtility
         /// Processes the parameters to return a valid resource Id.
         /// </summary>
         /// <param name="resourceId">The resource Id.</param>
-        /// <param name="resourceGroupName">The resource group</param>
         /// <param name="extensionResourceType">The extension resource type string in the format: '{providerName}/{typeName}/{nestedTypeName}'</param>
         /// <param name="extensionResourceName">The extension resource name in the format: '{resourceName}[/{nestedResourceName}]'</param>
         public static string GetResourceId(string resourceId, string extensionResourceType, string extensionResourceName = null)
@@ -357,7 +356,7 @@ public static (string scope, string relativeResourceId) SplitResourceId(string f
         }
 
         /// <summary>
-        /// Gets either a resource type or resource Id based on the value of the <see cref="getResourceName"/> parameter.
+        /// Gets either a resource type or resource Id based on the value of the <see cref="GetExtensionResourceTypeOrName"/> parameter.
         /// </summary>
         /// <param name="resourceId">The resource Id.</param>
         /// <param name="getResourceName">When set to true returns a resource name, otherwise a resource type.</param>
@@ -381,7 +380,7 @@ private static bool IsExtensionResourceId(string resourceId)
         }
 
         /// <summary>
-        /// Gets either a resource type or resource Id based on the value of the <see cref="getResourceName"/> parameter.
+        /// Gets either a resource type or resource Id based on the value of the <see cref="GetResourceTypeOrName"/> parameter.
         /// </summary>
         /// <param name="resourceId">The resource Id.</param>
         /// <param name="getResourceName">When set to true returns a resource name, otherwise a resource type.</param>
diff --git a/src/Resources/ResourceManager/Components/ResourceManagerClientHelper.cs b/src/Resources/ResourceManager/Components/ResourceManagerClientHelper.cs
index f9396920ec22..5e0b4f4eda5c 100644
--- a/src/Resources/ResourceManager/Components/ResourceManagerClientHelper.cs
+++ b/src/Resources/ResourceManager/Components/ResourceManagerClientHelper.cs
@@ -31,6 +31,7 @@ internal static class ResourceManagerClientHelper
         /// Gets a new instance of the <see cref="ResourceManagerRestRestClient"/>.
         /// </summary>
         /// <param name="context">The azure profile.</param>
+        /// <param name="cmdletHeaderValues"></param>
         internal static ResourceManagerRestRestClient GetResourceManagerClient(IAzureContext context, Dictionary<string, string> cmdletHeaderValues = null)
         {
             var endpoint = context.Environment.GetEndpoint(AzureEnvironment.Endpoint.ResourceManager);
diff --git a/src/Resources/ResourceManager/Components/TemplateSpecPackagingEngine.cs b/src/Resources/ResourceManager/Components/TemplateSpecPackagingEngine.cs
index 28a96b87e7cd..679b05786ab7 100644
--- a/src/Resources/ResourceManager/Components/TemplateSpecPackagingEngine.cs
+++ b/src/Resources/ResourceManager/Components/TemplateSpecPackagingEngine.cs
@@ -300,6 +300,7 @@ public static void Unpack(
         /// Gets all of the deployment resource JObjects within the specified template
         /// JObject.
         /// </summary>
+        /// <param name="templateObj"></param>
         /// <param name="includeNested">If true, deployment resource objects from nested
         /// templates will be included in the results</param>
         /// <remarks>
@@ -336,6 +337,7 @@ private static JObject[] GetDeploymentResourceObjects(JObject templateObj, bool
         /// Gets all of the TemplateLink JObjects within the specified template object that
         /// are representing references to Template Spec artifacts.
         /// </summary>
+        /// <param name="templateObj"></param>
         /// <param name="includeNested">If true, template links from nested templates 
         /// will be included in the results</param>
         /// <remarks>
diff --git a/src/Resources/ResourceManager/Extensions/ErrorResponseMessageExceptionExtensions.cs b/src/Resources/ResourceManager/Extensions/ErrorResponseMessageExceptionExtensions.cs
index e66230406195..90a3ce0edd82 100644
--- a/src/Resources/ResourceManager/Extensions/ErrorResponseMessageExceptionExtensions.cs
+++ b/src/Resources/ResourceManager/Extensions/ErrorResponseMessageExceptionExtensions.cs
@@ -46,7 +46,6 @@ internal static ErrorRecord ToErrorRecord(this Exception exception)
         /// <summary>
         /// Converts <see cref="AggregateException"/> objects into <see cref="ErrorRecord"/>
         /// </summary>
-        /// <param name="exception">The exception</param>
         internal static ErrorRecord ToErrorRecord(this AggregateException aggregateException)
         {
             // TODO: Improve this.
diff --git a/src/Resources/ResourceManager/Extensions/PsObjectExtensions.cs b/src/Resources/ResourceManager/Extensions/PsObjectExtensions.cs
index 3acb825090b1..d76f1c62c9cb 100644
--- a/src/Resources/ResourceManager/Extensions/PsObjectExtensions.cs
+++ b/src/Resources/ResourceManager/Extensions/PsObjectExtensions.cs
@@ -25,7 +25,7 @@ namespace Microsoft.Azure.Commands.ResourceManager.Cmdlets.Extensions
     using System.Management.Automation;
 
     /// <summary>
-    /// A helper class that handles common tasks that deal with the <see cref="ResourcePropertyObject"/> class.
+    /// A helper class that handles common tasks that deal with the <see cref="PsObjectExtensions"/> class.
     /// </summary>
     public static class PsObjectExtensions
     {
@@ -46,9 +46,9 @@ public static class PsObjectExtensions
         };
 
         /// <summary>
-        /// Converts a <see cref="ResourcePropertyObject"/> to <see cref="JToken"/>
+        /// Converts a <see cref="ToResourcePropertiesBody"/> to <see cref="JToken"/>
         /// </summary>
-        /// <param name="propertyObject">The <see cref="ResourcePropertyObject"/></param>
+        /// <param name="propertyObject">The <see cref="ToResourcePropertiesBody"/></param>
         internal static JToken ToResourcePropertiesBody(this PSObject propertyObject)
         {
             return PsObjectExtensions.ToJToken(propertyObject);
diff --git a/src/Resources/ResourceManager/Handlers/CmdletInfoHandler.cs b/src/Resources/ResourceManager/Handlers/CmdletInfoHandler.cs
index 96f44e4dc36c..bb861c5a5fc1 100644
--- a/src/Resources/ResourceManager/Handlers/CmdletInfoHandler.cs
+++ b/src/Resources/ResourceManager/Handlers/CmdletInfoHandler.cs
@@ -32,7 +32,7 @@ public class CmdletInfoHandler : DelegatingHandler
         /// <summary>
         /// Initializes a new instance of the <see cref="CmdletInfoHandler" /> class.
         /// </summary>
-        /// <param name="headerValues">The product info to add as headers.</param>
+        /// <param name="cmdletHeaderValues"></param>
         public CmdletInfoHandler(Dictionary<string, string> cmdletHeaderValues)
         {
             this.cmdletHeaderValues = cmdletHeaderValues;
diff --git a/src/Resources/ResourceManager/Implementation/Features/UnregisterAzureProviderPreviewFeatureCmdlet.cs b/src/Resources/ResourceManager/Implementation/Features/UnregisterAzureProviderPreviewFeatureCmdlet.cs
index 004e2067d2cc..7b63be256a45 100644
--- a/src/Resources/ResourceManager/Implementation/Features/UnregisterAzureProviderPreviewFeatureCmdlet.cs
+++ b/src/Resources/ResourceManager/Implementation/Features/UnregisterAzureProviderPreviewFeatureCmdlet.cs
@@ -39,9 +39,9 @@ public class UnregisterAzureProviderPreviewFeatureCmdlet : ProviderFeatureCmdlet
         public string ProviderNamespace { get; set; }
 
         [Parameter(Mandatory = false)]
-        /// <summary>
-        /// Gets or sets the pass thru.
-        /// </summary>
+        // <summary>
+        // Gets or sets the pass thru.
+        // </summary>
         public SwitchParameter PassThru { get; set; }
 
         /// <summary>
diff --git a/src/Resources/ResourceManager/Implementation/ManagedApplications/ManagedApplicationCmdletBase.cs b/src/Resources/ResourceManager/Implementation/ManagedApplications/ManagedApplicationCmdletBase.cs
index 43111763984a..9724d3ca7e9b 100644
--- a/src/Resources/ResourceManager/Implementation/ManagedApplications/ManagedApplicationCmdletBase.cs
+++ b/src/Resources/ResourceManager/Implementation/ManagedApplications/ManagedApplicationCmdletBase.cs
@@ -49,6 +49,7 @@ protected Task<ResponseWithContinuation<TType[]>> GetNextLink<TType>(string next
         /// <summary>
         /// Converts the resource object to specified resource type object.
         /// </summary>
+        /// <param name="resourceType"></param>
         /// <param name="resources">The policy definition resource object.</param>
         protected PSObject[] GetOutputObjects(string resourceType, params JToken[] resources)
         {
diff --git a/src/Resources/ResourceManager/Implementation/Policy/PolicyCmdletBase.cs b/src/Resources/ResourceManager/Implementation/Policy/PolicyCmdletBase.cs
index 9ee4c2349589..91abeb6accc6 100644
--- a/src/Resources/ResourceManager/Implementation/Policy/PolicyCmdletBase.cs
+++ b/src/Resources/ResourceManager/Implementation/Policy/PolicyCmdletBase.cs
@@ -70,7 +70,6 @@ public enum ListFilter
         /// <summary>
         /// Converts the resource object collection to a PsPolicyAssignment collection.
         /// </summary>
-        /// <param name="resourceType">The resource type of the objects to create</param>
         /// <param name="resources">The policy definition resource object.</param>
         protected PsPolicyAssignment[] GetOutputPolicyAssignments(params JToken[] resources)
         {
@@ -107,7 +106,6 @@ protected PsPolicySetDefinition[] GetOutputPolicySetDefinitions(params JToken[]
         /// <summary>
         /// Converts the resource object collection to a PsPolicyExemption collection.
         /// </summary>
-        /// <param name="resourceType">The resource type of the objects to create</param>
         /// <param name="resources">The policy definition resource object.</param>
         protected PsPolicyExemption[] GetOutputPolicyExemptions(params JToken[] resources)
         {
diff --git a/src/Resources/ResourceManager/SdkClient/ResourceManagerSdkClient.cs b/src/Resources/ResourceManager/SdkClient/ResourceManagerSdkClient.cs
index 909c030b4cdb..b690b6379c69 100644
--- a/src/Resources/ResourceManager/SdkClient/ResourceManagerSdkClient.cs
+++ b/src/Resources/ResourceManager/SdkClient/ResourceManagerSdkClient.cs
@@ -1371,6 +1371,7 @@ public virtual PSWhatIfOperationResult ExecuteDeploymentWhatIf(PSDeploymentWhatI
         /// Executes deployment What-If at the specified scope.
         /// </summary>
         /// <param name="parameters"></param>
+        /// <param name="excludeChangeTypeNames"></param>
         /// <returns></returns>
         public virtual PSWhatIfOperationResult ExecuteDeploymentWhatIf(PSDeploymentWhatIfCmdletParameters parameters, string[] excludeChangeTypeNames)
         {
@@ -1673,7 +1674,6 @@ private void CancelDeploymentAtResourceGroup(List<PSDeployment> deployments, str
         /// Validates a given deployment.
         /// </summary>
         /// <param name="parameters">The deployment create options</param>
-        /// <param name="deploymentMode">The deployment mode</param>
         /// <returns>The validation errors if there's any, or empty list otherwise.</returns>
         public virtual List<PSResourceManagerError> ValidateDeployment(PSDeploymentCmdletParameters parameters)
         {
diff --git a/src/Resources/ResourceManager/SdkClient/TemplateSpecsSdkClient.cs b/src/Resources/ResourceManager/SdkClient/TemplateSpecsSdkClient.cs
index b55d3a973469..8d7b97ab9239 100644
--- a/src/Resources/ResourceManager/SdkClient/TemplateSpecsSdkClient.cs
+++ b/src/Resources/ResourceManager/SdkClient/TemplateSpecsSdkClient.cs
@@ -313,7 +313,7 @@ public void DeleteTemplateSpec(
         /// <remarks>
         /// Method name is protected and has an 'Internal' suffix because the return type is
         /// the SDK model rather than the model wrapped for PS. See
-        /// <see cref="CreateOrUpdateTemplateSpec(string, string, string, string, string)"/>
+        /// <see cref="CreateOrUpdateTemplateSpecInternal"/>
         /// for the method that returns the wrapped model.
         /// </remarks>
         protected TemplateSpec CreateOrUpdateTemplateSpecInternal(

From 8844125d3401daf3f5c87b8c745671fdf50d2c28 Mon Sep 17 00:00:00 2001
From: Ziyue Zheng <v-ziyzhe@microsoft.com>
Date: Tue, 21 Jun 2022 18:49:41 +0800
Subject: [PATCH 3/3] Fix Az.Resources XML Comments

---
 .../ResourceManager/Components/ApiVersionHelper.cs   |  6 +++---
 .../ResourceManager/Components/HttpClientHelper.cs   |  2 +-
 .../Components/HttpClientHelperFactory.cs            |  4 ++--
 .../ResourceManager/Components/ResourceIdUtility.cs  |  4 ++--
 .../Components/ResourceManagerClientHelper.cs        |  2 +-
 .../ResourceManager/Extensions/PsObjectExtensions.cs |  6 +++---
 .../ResourceManager/Handlers/CmdletInfoHandler.cs    |  2 +-
 .../UnregisterAzureProviderPreviewFeatureCmdlet.cs   |  6 +++---
 .../ManagedApplicationCmdletBase.cs                  |  2 +-
 .../SdkClient/TemplateSpecsSdkClient.cs              |  2 +-
 .../Models.Authorization/AuthorizationClient.cs      | 12 ++++++------
 11 files changed, 24 insertions(+), 24 deletions(-)

diff --git a/src/Resources/ResourceManager/Components/ApiVersionHelper.cs b/src/Resources/ResourceManager/Components/ApiVersionHelper.cs
index 671959597508..894c52279f5d 100644
--- a/src/Resources/ResourceManager/Components/ApiVersionHelper.cs
+++ b/src/Resources/ResourceManager/Components/ApiVersionHelper.cs
@@ -46,7 +46,7 @@ internal static class ApiVersionHelper
         /// <param name="resourceId">The resource Id.</param>
         /// <param name="cancellationToken">The cancellation token</param>
         /// <param name="pre">When specified, indicates if pre-release API versions should be considered.</param>
-        /// <param name="cmdletHeaderValues"></param>
+        /// <param name="cmdletHeaderValues">The cmdlet info header values.</param>
         internal static Task<string> DetermineApiVersion(IAzureContext context, string resourceId, CancellationToken cancellationToken, bool? pre = null, Dictionary<string, string> cmdletHeaderValues = null)
         {
             var providerNamespace = ResourceIdUtility.GetExtensionProviderNamespace(resourceId)
@@ -66,7 +66,7 @@ internal static Task<string> DetermineApiVersion(IAzureContext context, string r
         /// <param name="resourceType">The resource type.</param>
         /// <param name="cancellationToken">The cancellation token</param>
         /// <param name="pre">When specified, indicates if pre-release API versions should be considered.</param>
-        /// <param name="cmdletHeaderValues"></param>
+        /// <param name="cmdletHeaderValues">The cmdlet info header values.</param>
         internal static Task<string> DetermineApiVersion(IAzureContext context, string providerNamespace, string resourceType, CancellationToken cancellationToken, bool? pre = null, Dictionary<string, string> cmdletHeaderValues = null)
         {
             var cacheKey = ApiVersionCache.GetCacheKey(context.Environment.Name, providerNamespace: providerNamespace, resourceType: resourceType);
@@ -108,7 +108,7 @@ internal static Task<string> DetermineApiVersion(IAzureContext context, string p
         /// <param name="providerNamespace">The provider namespace.</param>
         /// <param name="resourceType">The resource type.</param>
         /// <param name="cancellationToken">The cancellation token.</param>
-        /// <param name="cmdletHeaderValues"></param>
+        /// <param name="cmdletHeaderValues">The cmdlet info header values.</param>
         private static string[] GetApiVersionsForResourceType(IAzureContext context, string providerNamespace, string resourceType, CancellationToken cancellationToken, Dictionary<string, string> cmdletHeaderValues = null)
         {
             var resourceManagerClient = ResourceManagerClientHelper.GetResourceManagerClient(context, cmdletHeaderValues);
diff --git a/src/Resources/ResourceManager/Components/HttpClientHelper.cs b/src/Resources/ResourceManager/Components/HttpClientHelper.cs
index 67534a5d14a2..8dbf1644a2cb 100644
--- a/src/Resources/ResourceManager/Components/HttpClientHelper.cs
+++ b/src/Resources/ResourceManager/Components/HttpClientHelper.cs
@@ -47,7 +47,7 @@ public abstract class HttpClientHelper
         /// </summary>
         /// <param name="credentials">The service client credentials.</param>
         /// <param name="headerValues">The header values.</param>
-        /// <param name="cmdletHeaderValues"></param>
+        /// <param name="cmdletHeaderValues">The cmdlet info header values.</param>
         protected HttpClientHelper(ServiceClientCredentials credentials, IEnumerable<ProductInfoHeaderValue> headerValues, Dictionary<string, string> cmdletHeaderValues)
         {
             this.credentials = credentials;
diff --git a/src/Resources/ResourceManager/Components/HttpClientHelperFactory.cs b/src/Resources/ResourceManager/Components/HttpClientHelperFactory.cs
index 584fd17c4667..b4d96d8a1952 100644
--- a/src/Resources/ResourceManager/Components/HttpClientHelperFactory.cs
+++ b/src/Resources/ResourceManager/Components/HttpClientHelperFactory.cs
@@ -48,7 +48,7 @@ protected HttpClientHelperFactory()
         /// </summary>
         /// <param name="credentials">The credentials.</param>
         /// <param name="headerValues">The headers.</param>
-        /// <param name="cmdletHeaderValues"></param>
+        /// <param name="cmdletHeaderValues">The cmdlet info header.</param>
         public virtual HttpClientHelper CreateHttpClientHelper(ServiceClientCredentials credentials, IEnumerable<ProductInfoHeaderValue> headerValues, Dictionary<string, string> cmdletHeaderValues)
         {
             return new HttpClientHelperImpl(credentials: credentials, headerValues: headerValues, cmdletHeaderValues: cmdletHeaderValues);
@@ -64,7 +64,7 @@ private class HttpClientHelperImpl : HttpClientHelper
             /// </summary>
             /// <param name="credentials">The credentials.</param>
             /// <param name="headerValues">The headers.</param>
-            /// <param name="cmdletHeaderValues"></param>
+            /// <param name="cmdletHeaderValues">The cmdlet info header.</param>
             public HttpClientHelperImpl(ServiceClientCredentials credentials, IEnumerable<ProductInfoHeaderValue> headerValues, Dictionary<string, string> cmdletHeaderValues)
                 : base(credentials: credentials, headerValues: headerValues, cmdletHeaderValues: cmdletHeaderValues)
             {
diff --git a/src/Resources/ResourceManager/Components/ResourceIdUtility.cs b/src/Resources/ResourceManager/Components/ResourceIdUtility.cs
index db1f43ccf738..7d7e7ccf60b0 100644
--- a/src/Resources/ResourceManager/Components/ResourceIdUtility.cs
+++ b/src/Resources/ResourceManager/Components/ResourceIdUtility.cs
@@ -356,7 +356,7 @@ public static (string scope, string relativeResourceId) SplitResourceId(string f
         }
 
         /// <summary>
-        /// Gets either a resource type or resource Id based on the value of the <see cref="GetExtensionResourceTypeOrName"/> parameter.
+        /// Gets either a resource type or resource Id based on the value of the <paramref name="getResourceName"/> parameter.
         /// </summary>
         /// <param name="resourceId">The resource Id.</param>
         /// <param name="getResourceName">When set to true returns a resource name, otherwise a resource type.</param>
@@ -380,7 +380,7 @@ private static bool IsExtensionResourceId(string resourceId)
         }
 
         /// <summary>
-        /// Gets either a resource type or resource Id based on the value of the <see cref="GetResourceTypeOrName"/> parameter.
+        /// Gets either a resource type or resource Id based on the value of the <paramref name="getResourceName"/> parameter.
         /// </summary>
         /// <param name="resourceId">The resource Id.</param>
         /// <param name="getResourceName">When set to true returns a resource name, otherwise a resource type.</param>
diff --git a/src/Resources/ResourceManager/Components/ResourceManagerClientHelper.cs b/src/Resources/ResourceManager/Components/ResourceManagerClientHelper.cs
index 5e0b4f4eda5c..2583c58145a3 100644
--- a/src/Resources/ResourceManager/Components/ResourceManagerClientHelper.cs
+++ b/src/Resources/ResourceManager/Components/ResourceManagerClientHelper.cs
@@ -31,7 +31,7 @@ internal static class ResourceManagerClientHelper
         /// Gets a new instance of the <see cref="ResourceManagerRestRestClient"/>.
         /// </summary>
         /// <param name="context">The azure profile.</param>
-        /// <param name="cmdletHeaderValues"></param>
+        /// <param name="cmdletHeaderValues">The cmdlet info header values.</param>
         internal static ResourceManagerRestRestClient GetResourceManagerClient(IAzureContext context, Dictionary<string, string> cmdletHeaderValues = null)
         {
             var endpoint = context.Environment.GetEndpoint(AzureEnvironment.Endpoint.ResourceManager);
diff --git a/src/Resources/ResourceManager/Extensions/PsObjectExtensions.cs b/src/Resources/ResourceManager/Extensions/PsObjectExtensions.cs
index d76f1c62c9cb..abcf785e79e3 100644
--- a/src/Resources/ResourceManager/Extensions/PsObjectExtensions.cs
+++ b/src/Resources/ResourceManager/Extensions/PsObjectExtensions.cs
@@ -25,7 +25,7 @@ namespace Microsoft.Azure.Commands.ResourceManager.Cmdlets.Extensions
     using System.Management.Automation;
 
     /// <summary>
-    /// A helper class that handles common tasks that deal with the <see cref="PsObjectExtensions"/> class.
+    /// A helper class that handles common tasks that deal with the <see cref="PSObject"/> class.
     /// </summary>
     public static class PsObjectExtensions
     {
@@ -46,9 +46,9 @@ public static class PsObjectExtensions
         };
 
         /// <summary>
-        /// Converts a <see cref="ToResourcePropertiesBody"/> to <see cref="JToken"/>
+        /// Converts a <see cref="PSObject"/> to <see cref="JToken"/>
         /// </summary>
-        /// <param name="propertyObject">The <see cref="ToResourcePropertiesBody"/></param>
+        /// <param name="propertyObject">The ResourcePropertyObject.</param>
         internal static JToken ToResourcePropertiesBody(this PSObject propertyObject)
         {
             return PsObjectExtensions.ToJToken(propertyObject);
diff --git a/src/Resources/ResourceManager/Handlers/CmdletInfoHandler.cs b/src/Resources/ResourceManager/Handlers/CmdletInfoHandler.cs
index bb861c5a5fc1..aaad766b1976 100644
--- a/src/Resources/ResourceManager/Handlers/CmdletInfoHandler.cs
+++ b/src/Resources/ResourceManager/Handlers/CmdletInfoHandler.cs
@@ -32,7 +32,7 @@ public class CmdletInfoHandler : DelegatingHandler
         /// <summary>
         /// Initializes a new instance of the <see cref="CmdletInfoHandler" /> class.
         /// </summary>
-        /// <param name="cmdletHeaderValues"></param>
+        /// <param name="cmdletHeaderValues">The product info to add as headers.</param>
         public CmdletInfoHandler(Dictionary<string, string> cmdletHeaderValues)
         {
             this.cmdletHeaderValues = cmdletHeaderValues;
diff --git a/src/Resources/ResourceManager/Implementation/Features/UnregisterAzureProviderPreviewFeatureCmdlet.cs b/src/Resources/ResourceManager/Implementation/Features/UnregisterAzureProviderPreviewFeatureCmdlet.cs
index 7b63be256a45..d2201b982588 100644
--- a/src/Resources/ResourceManager/Implementation/Features/UnregisterAzureProviderPreviewFeatureCmdlet.cs
+++ b/src/Resources/ResourceManager/Implementation/Features/UnregisterAzureProviderPreviewFeatureCmdlet.cs
@@ -38,10 +38,10 @@ public class UnregisterAzureProviderPreviewFeatureCmdlet : ProviderFeatureCmdlet
         [ValidateNotNullOrEmpty]
         public string ProviderNamespace { get; set; }
 
+        /// <summary>
+        /// Gets or sets the pass thru.
+        /// </summary>
         [Parameter(Mandatory = false)]
-        // <summary>
-        // Gets or sets the pass thru.
-        // </summary>
         public SwitchParameter PassThru { get; set; }
 
         /// <summary>
diff --git a/src/Resources/ResourceManager/Implementation/ManagedApplications/ManagedApplicationCmdletBase.cs b/src/Resources/ResourceManager/Implementation/ManagedApplications/ManagedApplicationCmdletBase.cs
index 9724d3ca7e9b..59bf0b95ca73 100644
--- a/src/Resources/ResourceManager/Implementation/ManagedApplications/ManagedApplicationCmdletBase.cs
+++ b/src/Resources/ResourceManager/Implementation/ManagedApplications/ManagedApplicationCmdletBase.cs
@@ -49,7 +49,7 @@ protected Task<ResponseWithContinuation<TType[]>> GetNextLink<TType>(string next
         /// <summary>
         /// Converts the resource object to specified resource type object.
         /// </summary>
-        /// <param name="resourceType"></param>
+        /// <param name="resourceType">The resource type.</param>
         /// <param name="resources">The policy definition resource object.</param>
         protected PSObject[] GetOutputObjects(string resourceType, params JToken[] resources)
         {
diff --git a/src/Resources/ResourceManager/SdkClient/TemplateSpecsSdkClient.cs b/src/Resources/ResourceManager/SdkClient/TemplateSpecsSdkClient.cs
index 8d7b97ab9239..f4834098ccfa 100644
--- a/src/Resources/ResourceManager/SdkClient/TemplateSpecsSdkClient.cs
+++ b/src/Resources/ResourceManager/SdkClient/TemplateSpecsSdkClient.cs
@@ -313,7 +313,7 @@ public void DeleteTemplateSpec(
         /// <remarks>
         /// Method name is protected and has an 'Internal' suffix because the return type is
         /// the SDK model rather than the model wrapped for PS. See
-        /// <see cref="CreateOrUpdateTemplateSpecInternal"/>
+        /// <see cref="CreateOrUpdateTemplateSpec"/>
         /// for the method that returns the wrapped model.
         /// </remarks>
         protected TemplateSpec CreateOrUpdateTemplateSpecInternal(
diff --git a/src/Resources/Resources/Models.Authorization/AuthorizationClient.cs b/src/Resources/Resources/Models.Authorization/AuthorizationClient.cs
index 9ddd87055755..781fe413196f 100644
--- a/src/Resources/Resources/Models.Authorization/AuthorizationClient.cs
+++ b/src/Resources/Resources/Models.Authorization/AuthorizationClient.cs
@@ -61,7 +61,7 @@ public PSRoleDefinition GetRoleDefinition(string roleId)
         /// Gets a single role definition by the role Id guid.
         /// </summary>
         /// <param name="roleId">RoleId guid</param>
-        /// <param name="scope"></param>
+        /// <param name="scope">The scope of the role definition</param>
         public PSRoleDefinition GetRoleDefinition(Guid roleId, string scope)
         {
             return AuthorizationManagementClient.RoleDefinitions.Get(scope, roleId.ToString()).ToPSRoleDefinition();
@@ -73,7 +73,7 @@ public PSRoleDefinition GetRoleDefinition(Guid roleId, string scope)
         /// Otherwise  will fetch Roledefinitions with provided name
         /// </summary>
         /// <param name="name">The role name</param>
-        /// <param name="scope"></param>
+        /// <param name="scope">The scope of the role definition</param>
         /// <param name="first"></param>
         /// <param name="skip"></param>
         /// <returns>The matched role Definitions</returns>
@@ -129,7 +129,7 @@ public IEnumerable<PSRoleDefinition> FilterRoleDefinitionsByCustom(string scope,
         /// Creates new role assignment.
         /// </summary>
         /// <param name="parameters">The create parameters</param>
-        /// <param name="roleAssignmentId"></param>
+        /// <param name="roleAssignmentId">The name of the role assignment. It can be any valid GUID.</param>
         /// <returns>The created role assignment object</returns>
         public PSRoleAssignment CreateRoleAssignment(FilterRoleAssignmentsOptions parameters, Guid roleAssignmentId = default(Guid))
         {
@@ -389,7 +389,7 @@ public PSRoleDefinition GetSingleRoleDefinitionByName(string name, string scope)
         /// Deletes a role definition based on the id.
         /// </summary>
         /// <param name="roleDefinitionId">The role definition id to delete</param>
-        /// <param name="scope"></param>
+        /// <param name="scope">The scope of the role definition</param>
         /// <returns>The deleted role definition.</returns>
         public PSRoleDefinition RemoveRoleDefinition(Guid roleDefinitionId, string scope)
         {
@@ -408,7 +408,7 @@ public PSRoleDefinition RemoveRoleDefinition(Guid roleDefinitionId, string scope
         /// Deletes a role definition based on the name.
         /// </summary>
         /// <param name="roleDefinitionName">The role definition name.</param>
-        /// <param name="scope"></param>
+        /// <param name="scope">The scope of the role definition</param>
         /// <returns>The deleted role definition.</returns>
         public PSRoleDefinition RemoveRoleDefinition(string roleDefinitionName, string scope)
         {
@@ -475,7 +475,7 @@ public PSRoleDefinition UpdateRoleDefinition(PSRoleDefinition roleDefinition)
         /// Creates a new role definition.
         /// </summary>
         /// <param name="roleDefinition">The role definition to create.</param>
-        /// <param name="roleDefinitionId">The role definition id to delete</param>
+        /// <param name="roleDefinitionId">The role definition id to create.</param>
         /// <returns>The created role definition.</returns>
         public PSRoleDefinition CreateRoleDefinition(PSRoleDefinition roleDefinition, Guid roleDefinitionId = default(Guid))
         {