diff --git a/src/RecoveryServices/RecoveryServices.Backup.Models/AzureVmWorkloadModels/AzureWorkloadProtectableItem.cs b/src/RecoveryServices/RecoveryServices.Backup.Models/AzureVmWorkloadModels/AzureWorkloadProtectableItem.cs
index 3b2e95b2df89..af1fecc054cc 100644
--- a/src/RecoveryServices/RecoveryServices.Backup.Models/AzureVmWorkloadModels/AzureWorkloadProtectableItem.cs
+++ b/src/RecoveryServices/RecoveryServices.Backup.Models/AzureVmWorkloadModels/AzureWorkloadProtectableItem.cs
@@ -93,7 +93,7 @@ public class AzureWorkloadProtectableItem : ProtectableItemBase
         /// Constructor. Takes the service client object representing the protected item 
         /// and converts it in to the PS protected item model
         /// </summary>
-        /// <param name="workloadProtectableItemResource"></param>
+        /// <param name="workloadProtectableItemResource">Service client object representing the protected item resource</param>
         /// <param name="containerName">Name of the container associated with this protected item</param>
         /// <param name="containerType">Type of the container associated with this protected item</param>
         public AzureWorkloadProtectableItem(WorkloadProtectableItemResource workloadProtectableItemResource,
@@ -111,7 +111,7 @@ public AzureWorkloadProtectableItem(WorkloadProtectableItemResource workloadProt
             Subinquireditemcount = protectedItem.Subinquireditemcount;
             Subprotectableitemcount = protectedItem.Subprotectableitemcount;
             Prebackupvalidation = protectedItem.Prebackupvalidation;
-            ProtectableItemType = workloadProtectableItemResource.Properties.GetType().ToString();            
+            ProtectableItemType = workloadProtectableItemResource.Properties.GetType().ToString();
 
             if (workloadProtectableItemResource.Properties.GetType() == typeof(AzureVmWorkloadSQLAvailabilityGroupProtectableItem))
             {
@@ -124,7 +124,7 @@ public AzureWorkloadProtectableItem(WorkloadProtectableItemResource workloadProt
             else if (workloadProtectableItemResource.Properties.GetType() == typeof(AzureVmWorkloadSQLDatabaseProtectableItem))
             {
                 ProtectableItemType = CmdletModel.ProtectableItemType.SQLDataBase.ToString();
-            }            
+            }
         }
     }
 }
\ No newline at end of file
diff --git a/src/RecoveryServices/RecoveryServices.Backup.Providers/AzureWorkloadProviderHelper.cs b/src/RecoveryServices/RecoveryServices.Backup.Providers/AzureWorkloadProviderHelper.cs
index a8fd8d70dd3e..a5277d59dbe0 100644
--- a/src/RecoveryServices/RecoveryServices.Backup.Providers/AzureWorkloadProviderHelper.cs
+++ b/src/RecoveryServices/RecoveryServices.Backup.Providers/AzureWorkloadProviderHelper.cs
@@ -532,7 +532,7 @@ public List<SubProtectionPolicy> GetSubProtectionPolicyOfType(IList<SubProtectio
         {
             return newSubProtectionPolicies.Where(
                 newSubProtectionPolicy =>
-                {                   
+                {
                     return (newSubProtectionPolicy.PolicyType == policyType);
                 }).ToList();
         }
@@ -546,8 +546,8 @@ public bool checkInstantRpRetentionRange(int? oldSnapshotRetention, int? newSnap
         /// checks if daily retention is reduced first, then weekly, then monthly and then yearly; breaks and return true whenever it finds retention is reduced in any schedule.
         /// if retention is not reduced in any schedule, returns false at the end.
         /// </summary>
-        /// <param name="newRetentionPolicy"></param>
         /// <param name="oldRetentionPolicy"></param>
+        /// <param name="newRetentionPolicy"></param>
         /// <returns></returns>
         public bool checkMUAForLongTermRetentionPolicy(ServiceClientModel.LongTermRetentionPolicy oldRetentionPolicy, ServiceClientModel.LongTermRetentionPolicy newRetentionPolicy)
         {
@@ -578,7 +578,7 @@ public bool checkMUAForLongTermRetentionPolicy(ServiceClientModel.LongTermRetent
                 {
                     return true;
                 }
-            }            
+            }
 
             return false;
         }
@@ -600,13 +600,13 @@ public bool checkMUAForSimpleRetentionPolicy(ServiceClientModel.SimpleRetentionP
         /// <param name="newRetentionPolicy"></param>
         /// <returns></returns>
         public bool checkMUAForMSSQLPolicy(ServiceClientModel.AzureVmWorkloadProtectionPolicy oldRetentionPolicy, ServiceClientModel.AzureVmWorkloadProtectionPolicy newRetentionPolicy)
-        {            
+        {
             IList<SubProtectionPolicy> oldSubProtectionPolicies = oldRetentionPolicy.SubProtectionPolicy;
             IList<SubProtectionPolicy> newSubProtectionPolicies = newRetentionPolicy.SubProtectionPolicy;
 
             foreach (SubProtectionPolicy oldSubProtectionPolicy in oldSubProtectionPolicies)
             {
-                string policyType = oldSubProtectionPolicy.PolicyType;                
+                string policyType = oldSubProtectionPolicy.PolicyType;
                 List<SubProtectionPolicy> newSubProtectionPolicy = GetSubProtectionPolicyOfType(newSubProtectionPolicies, policyType);
                 if(newSubProtectionPolicy == null || newSubProtectionPolicy.Count == 0) return true;
                 else
@@ -616,7 +616,7 @@ public bool checkMUAForMSSQLPolicy(ServiceClientModel.AzureVmWorkloadProtectionP
                         if(checkMUAForSimpleRetentionPolicy((ServiceClientModel.SimpleRetentionPolicy)oldSubProtectionPolicy.RetentionPolicy, (ServiceClientModel.SimpleRetentionPolicy)newSubProtectionPolicy[0].RetentionPolicy))
                         {
                             return true;
-                        }                        
+                        }
                     }
                     else if (oldSubProtectionPolicy.RetentionPolicy.GetType() == typeof(ServiceClientModel.LongTermRetentionPolicy))
                     {
@@ -654,7 +654,7 @@ public bool checkMUAForRetentionPolicy(ProtectionPolicyResource oldPolicy, Prote
             }
 
             else if (newPolicy.Properties.GetType() == typeof(ServiceClientModel.AzureVmWorkloadProtectionPolicy))
-            {                   
+            {
                 return checkMUAForMSSQLPolicy((ServiceClientModel.AzureVmWorkloadProtectionPolicy)oldPolicy.Properties, (ServiceClientModel.AzureVmWorkloadProtectionPolicy)newPolicy.Properties);                                
             }
 
@@ -662,7 +662,7 @@ public bool checkMUAForRetentionPolicy(ProtectionPolicyResource oldPolicy, Prote
         }
 
         public bool checkMUAForModifyPolicy(ProtectionPolicyResource oldPolicy, ProtectionPolicyResource newPolicy, bool enableMUA = false)
-        {            
+        {
             if( enableMUA && (checkMUAForSchedulePolicy(oldPolicy, newPolicy) || checkMUAForRetentionPolicy(oldPolicy, newPolicy)))
             {
                 return true;
@@ -706,7 +706,7 @@ public void CopyScheduleTimeToRetentionTimes(CmdletModel.LongTermRetentionPolicy
 
                 // schedule runTimes is already validated if in UTC/not during validate()
                 // now copy times from schedule to retention policy
-                
+
                 List<DateTime> hourlyWindowStartTime = (schPolicyV2.HourlySchedule != null) ? new List<DateTime>{(DateTime)schPolicyV2.HourlySchedule.WindowStartTime} : null;
 
                 if (retPolicy.IsDailyScheduleEnabled && retPolicy.DailySchedule != null)
diff --git a/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/ItemAPIs.cs b/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/ItemAPIs.cs
index 8adf8ffacbf5..a4ab3283f7af 100644
--- a/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/ItemAPIs.cs
+++ b/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/ItemAPIs.cs
@@ -73,7 +73,7 @@ public RestAzureNS.AzureOperationResponse<ProtectedItemResource> CreateOrUpdateP
                         customHeaders.Add("x-ms-authorization-auxiliary", new List<string> { "Bearer " + auxiliaryAccessToken });
                     }
                     else
-                    {                        
+                    {
                         throw new ArgumentException(String.Format(Resources.UnexpectedParameterToken, "ModifyProtection"));
                     }
                 }
@@ -116,25 +116,25 @@ public RestAzureNS.AzureOperationResponse DeleteProtectedItem(
 
             string operationRequest = null;
 
-            // unlock 
-            UnlockDeleteRequest unlockDeleteRequest = new UnlockDeleteRequest();                       
+            // unlock
+            UnlockDeleteRequest unlockDeleteRequest = new UnlockDeleteRequest();
 
             List<ResourceGuardProxyBaseResource> resourceGuardMapping = ListResourceGuardMapping(vaultName, resourceGroupName);
 
             if (resourceGuardMapping != null && resourceGuardMapping.Count != 0)
-            {                
+            {
                 foreach (ResourceGuardOperationDetail operationDetail in resourceGuardMapping[0].Properties.ResourceGuardOperationDetails)
                 {
                     if (operationDetail.VaultCriticalOperation == "Microsoft.RecoveryServices/vaults/backupFabrics/protectionContainers/protectedItems/delete") operationRequest = operationDetail.DefaultResourceRequest;
                 }
-                               
+
                 if(operationRequest != null)
                 {
                     unlockDeleteRequest.ResourceGuardOperationRequests = new List<string>();
                     unlockDeleteRequest.ResourceGuardOperationRequests.Add(operationRequest);
 
                     if(protectedItemUri == null)
-                    {                        
+                    {
                         throw new ArgumentException(String.Format(Resources.ProtectedItemURICantBeNull));
                     }
 
@@ -142,7 +142,7 @@ public RestAzureNS.AzureOperationResponse DeleteProtectedItem(
                     UnlockDeleteResponse unlockDeleteResponse = BmsAdapter.Client.ResourceGuardProxy.UnlockDeleteWithHttpMessagesAsync(vaultName ?? BmsAdapter.GetResourceName(), resourceGroupName ?? BmsAdapter.GetResourceGroupName(), resourceGuardMapping[0].Name, unlockDeleteRequest, customHeaders).Result.Body;
                 }
                 else if (auxiliaryAccessToken != null && auxiliaryAccessToken != "")
-                {   
+                {
                     throw new ArgumentException(String.Format(Resources.UnexpectedParameterToken, "Delete protection with DeleteBackupData"));
                 }
             }
@@ -156,7 +156,7 @@ public RestAzureNS.AzureOperationResponse DeleteProtectedItem(
                 resourceGroupName ?? BmsAdapter.GetResourceGroupName(),
                 AzureFabricName,
                 containerName,
-                protectedItemName,                
+                protectedItemName,
                 cancellationToken: BmsAdapter.CmdletCancellationToken).Result;
         }
 
@@ -231,7 +231,7 @@ public List<ProtectedItemResource> ListProtectedItem(
             string skipToken = default(string),
             string vaultName = null,
             string resourceGroupName = null)
-        {            
+        {
             Func<RestAzureNS.IPage<CrrModel.ProtectedItemResource>> listAsync =
                 () => CrrAdapter.Client.BackupProtectedItemsCrr.ListWithHttpMessagesAsync(
                     vaultName ?? BmsAdapter.GetResourceName(),
@@ -383,7 +383,7 @@ public List<ProtectionIntentResource> ListProtectionIntent(
         /// <summary>
         /// Lists workload items according to the query filter and the pagination params
         /// </summary>
-        /// <param name="containerName"></param>
+        /// <param name="containerName">Name of the container which this item belongs to</param>
         /// <param name="queryFilter">Query filter</param>
         /// <param name="skipToken">Skip token for pagination</param>
         /// <param name="vaultName"></param>
diff --git a/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/OperationStatusAPIs.cs b/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/OperationStatusAPIs.cs
index 677881b4da18..1adb70811025 100644
--- a/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/OperationStatusAPIs.cs
+++ b/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/OperationStatusAPIs.cs
@@ -44,7 +44,7 @@ public RestAzureNS.AzureOperationResponse<ServiceClientModel.OperationStatus>
         /// <summary>
         /// Gets status of a generic operation on the protected item using the operation ID
         /// </summary>
-        /// <param name="secondaryRegion"></param>
+        /// <param name="secondaryRegion">secondary region where to trigger the restore</param>
         /// <param name="operationId">ID of the operation in progress</param>
         /// <returns>Operation status response returned by the service</returns>
         public RestAzureNS.AzureOperationResponse<CrrModel.OperationStatus>
@@ -105,7 +105,7 @@ public RestAzureNS.AzureOperationResponse GetContainerRefreshOrInquiryOperationR
         /// Gets result of the refresh operation on the protection container using the operation ID
         /// </summary>
         /// <param name="operationId">ID of the operation in progress</param>
-        /// <param name="containerName"></param>
+        /// <param name="containerName">Name of the container which this item belongs to</param>
         /// <param name="vaultName"></param>
         /// <param name="resourceGroupName"></param>
         /// <returns></returns>
diff --git a/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/RecoveryPointsAPIs.cs b/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/RecoveryPointsAPIs.cs
index 6757ccc0046f..0a2ce2b5cf32 100644
--- a/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/RecoveryPointsAPIs.cs
+++ b/src/RecoveryServices/RecoveryServices.Backup.ServiceClientAdapter/BMSAPIs/RecoveryPointsAPIs.cs
@@ -127,7 +127,7 @@ public List<RecoveryPointResource> GetRecoveryPoints(
         /// </summary>
         /// <param name="containerName">Name of the container which the item belongs to</param>
         /// <param name="protectedItemName">Name of the item</param>
-        /// <param name="moveRequest"></param>
+        /// <param name="moveRequest">List Recovery points Recommended for Move Request</param>
         /// <param name="vaultName"></param>
         /// <param name="resourceGroupName"></param>
         /// <returns>List of recovery points</returns>
@@ -145,7 +145,7 @@ public List<RecoveryPointResource> GetMoveRecommendedRecoveryPoints(
                 AzureFabricName,
                 containerName,
                 protectedItemName,
-                moveRequest             
+                moveRequest
                 ).Result.Body;
 
             Func<string, RestAzureNS.IPage<RecoveryPointResource>> listNextAsync =
@@ -162,7 +162,7 @@ public List<RecoveryPointResource> GetMoveRecommendedRecoveryPoints(
         /// </summary>
         /// <param name="containerName">Name of the container which the item belongs to</param>
         /// <param name="protectedItemName">Name of the item</param>
-        /// <param name="moveRPAcrossTiersRequest"></param>
+        /// <param name="moveRPAcrossTiersRequest">Move Resource Across Tiers Request</param>
         /// <param name="recoveryPointId"></param>
         /// <param name="vaultName"></param>
         /// <param name="resourceGroupName"></param>
diff --git a/src/RecoveryServices/RecoveryServices.Backup/RSBackupVaultCmdletBase.cs b/src/RecoveryServices/RecoveryServices.Backup/RSBackupVaultCmdletBase.cs
index cf1a6da1c5c1..559e85b5f999 100644
--- a/src/RecoveryServices/RecoveryServices.Backup/RSBackupVaultCmdletBase.cs
+++ b/src/RecoveryServices/RecoveryServices.Backup/RSBackupVaultCmdletBase.cs
@@ -59,8 +59,8 @@ public CmdletModel.JobBase GetJobObject(string jobId, string vaultName = null, s
         /// <summary>
         /// Get the job PS model after fetching the job object from the service given the job ID.
         /// </summary>
-        /// <param name="secondaryRegion"></param>
-        /// <param name="vaultId"></param>
+        /// <param name="secondaryRegion">secondaryRegion for the vault </param>
+        /// <param name="vaultId">ResourceId of the vault to be fetched</param>
         /// <param name="jobId">ID of the job to be fetched</param>
         /// <returns></returns>
         public CmdletModel.JobBase GetCrrJobObject(string secondaryRegion, string vaultId, string jobId)
@@ -68,7 +68,7 @@ public CmdletModel.JobBase GetCrrJobObject(string secondaryRegion, string vaultI
             CrrModel.CrrJobRequest jobRequest = new CrrModel.CrrJobRequest();
             jobRequest.JobName = jobId;
             jobRequest.ResourceId = vaultId;
-                        
+
             JobBase job = JobConversions.GetPSJobCrr(ServiceClientAdapter.GetCRRJobDetails(
                 secondaryRegion,
                 jobRequest));
diff --git a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrClient.cs b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrClient.cs
index feca29838b58..7b8f8f11d922 100644
--- a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrClient.cs
+++ b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrClient.cs
@@ -82,7 +82,7 @@ public partial class PSRecoveryServicesClient
         ///     Initializes a new instance of the <see cref="PSRecoveryServicesClient" /> class with
         ///     required current subscription.
         /// </summary>
-        /// <param name="azureProfile"></param>
+        /// <param name="azureProfile">Azure context.</param>
         public PSRecoveryServicesClient(
             IAzureContextContainer azureProfile)
         {
diff --git a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrComputeManagementClient.cs b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrComputeManagementClient.cs
index d881f1750fd4..d821df729b2b 100644
--- a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrComputeManagementClient.cs
+++ b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrComputeManagementClient.cs
@@ -54,7 +54,7 @@ public partial class PSAsrComputeManagementClient
         ///     Initializes a new instance of the <see cref="PSRecoveryServicesClient" /> class with
         ///     required current subscription.
         /// </summary>
-        /// <param name="azureProfile"></param>
+        /// <param name="azureProfile">Azure context.</param>
         public PSAsrComputeManagementClient(
             IAzureContextContainer azureProfile)
         {
diff --git a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrEventClient.cs b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrEventClient.cs
index 435adaf58b8d..c0e82292ef2f 100644
--- a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrEventClient.cs
+++ b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrEventClient.cs
@@ -26,7 +26,7 @@ public partial class PSRecoveryServicesClient
         /// <summary>
         ///     Gets the events.
         /// </summary>
-        /// <param name="eventName"></param>
+        /// <param name="eventName">The name of the Azure Site Recovery event.</param>
         /// <returns></returns>
         public EventModel GetAzureRmSiteRecoveryEvent(string eventName)
         {
diff --git a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrNetworkClient.cs b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrNetworkClient.cs
index facafe08c2a5..a1b3c6e19052 100644
--- a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrNetworkClient.cs
+++ b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrNetworkClient.cs
@@ -68,7 +68,7 @@ public List<Network> GetAzureSiteRecoveryNetworks()
         /// <summary>
         ///     Gets all Azure Site Recovery Networks under a Server
         /// </summary>
-        /// <param name="fabricName"></param>
+        /// <param name="fabricName">Fabric name</param>
         /// <returns>Network list response</returns>
         public List<Network> GetAzureSiteRecoveryNetworks(
             string fabricName)
diff --git a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrNetworkMappingClient.cs b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrNetworkMappingClient.cs
index 3f64eaf2a0df..4c983495e146 100644
--- a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrNetworkMappingClient.cs
+++ b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrNetworkMappingClient.cs
@@ -105,7 +105,7 @@ public NetworkMapping GetAzureSiteRecoveryNetworkMappings(
         /// <param name="primaryFabricName">Primary fabric name</param>
         /// <param name="primaryNetworkName">Primary network name</param>
         /// <param name="mappingName">Mapping name</param>
-        /// <param name="input"></param>
+        /// <param name="input">Input data to be passed as request body.</param>
         /// <returns>Long running operation response</returns>
         public PSSiteRecoveryLongRunningOperation NewAzureSiteRecoveryNetworkMapping(
             string primaryFabricName,
diff --git a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrPolicyClient.cs b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrPolicyClient.cs
index 2c3b51f70645..a39a22840ed7 100644
--- a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrPolicyClient.cs
+++ b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrPolicyClient.cs
@@ -26,7 +26,7 @@ public partial class PSRecoveryServicesClient
         /// <summary>
         ///     Creates Azure Site Recovery Policy.
         /// </summary>
-        /// <param name="policyName">Policy name</param>
+        /// <param name="policyName">Policy Name</param>
         /// <param name="input">Policy Input</param>
         /// <returns>Long operation response</returns>
         public PSSiteRecoveryLongRunningOperation CreatePolicy(
@@ -47,7 +47,7 @@ public PSSiteRecoveryLongRunningOperation CreatePolicy(
         /// <summary>
         ///     Deletes Azure Site Recovery Policy.
         /// </summary>
-        /// <param name="policyName"></param>
+        /// <param name="policyName">Policy Name</param>
         /// <returns>Long operation response</returns>
         public PSSiteRecoveryLongRunningOperation DeletePolicy(
             string policyName)
diff --git a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrProtectionContainerClient.cs b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrProtectionContainerClient.cs
index 7cd63a80cfce..120805f72885 100644
--- a/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrProtectionContainerClient.cs
+++ b/src/RecoveryServices/RecoveryServices.SiteRecovery/Common/PSAsrProtectionContainerClient.cs
@@ -100,6 +100,7 @@ public List<ProtectionContainer> GetAzureSiteRecoveryProtectionContainer()
         /// <summary>
         ///     Gets Azure Site Recovery Protection Container.
         /// </summary>
+        /// <param name="fabricName">Fabric Name</param>
         /// <returns>Protection Container list response</returns>
         public List<ProtectionContainer> GetAzureSiteRecoveryProtectionContainer(
             string fabricName)
@@ -127,7 +128,7 @@ public List<ProtectionContainer> GetAzureSiteRecoveryProtectionContainer(
         /// <summary>
         ///     Gets Azure Site Recovery Protection Container.
         /// </summary>
-        /// <param name="fabricName"></param>
+        /// <param name="fabricName">Fabric Name</param>
         /// <param name="protectionContainerName">Protection Container ID</param>
         /// <returns>Protection Container response</returns>
         public ProtectionContainer GetAzureSiteRecoveryProtectionContainer(
diff --git a/src/RecoveryServices/RecoveryServices.SiteRecovery/Models/PSEvent.cs b/src/RecoveryServices/RecoveryServices.SiteRecovery/Models/PSEvent.cs
index da75e31cc5d7..4a42f32b5985 100644
--- a/src/RecoveryServices/RecoveryServices.SiteRecovery/Models/PSEvent.cs
+++ b/src/RecoveryServices/RecoveryServices.SiteRecovery/Models/PSEvent.cs
@@ -145,7 +145,7 @@ private IList<ASRHealthError> TranslateHealthErrors(IList<HealthError> healthErr
         /// <summary>
         ///     Translate Health errors to Powershell object.
         /// </summary>
-        /// <param name="eventSpecificDetails">Rest API ASREventSpecificDetails object.</param>
+        /// <param name="eventSpecificDetails">Rest API EventSpecificDetails object.</param>
         /// <returns></returns>
         private ASREventSpecificDetails TranslateEventSpecificDetails(
             EventSpecificDetails eventSpecificDetails)
@@ -160,7 +160,7 @@ private ASREventSpecificDetails TranslateEventSpecificDetails(
         /// <summary>
         ///     Translate Health errors to Powershell object.
         /// </summary>
-        /// <param name="eventSpecificDetails">Rest API ASREventSpecificDetails object.</param>
+        /// <param name="eventSpecificDetails">Rest API EventProviderSpecificDetails object.</param>
         /// <returns></returns>
         private ASREventProviderSpecificDetails TranslateProviderSpecificEventDetails(
             EventProviderSpecificDetails eventSpecificDetails)
@@ -319,7 +319,7 @@ public class ASRHyperVReplicaAzureEventDetails : ASREventProviderSpecificDetails
         /// <summary>
         ///     Converts REST API object to Powershell object.
         /// </summary>
-        /// <param name="eventDetails"></param>
+        /// <param name="eventDetails">Internal object for a monitoring event.</param>
         /// <returns>
         ///     REST API object for HyperVReplica E2A event provider specific
         ///     details.
diff --git a/src/RecoveryServices/RecoveryServices.SiteRecovery/Models/PSObjects.cs b/src/RecoveryServices/RecoveryServices.SiteRecovery/Models/PSObjects.cs
index af810cf5025b..f7117e766078 100644
--- a/src/RecoveryServices/RecoveryServices.SiteRecovery/Models/PSObjects.cs
+++ b/src/RecoveryServices/RecoveryServices.SiteRecovery/Models/PSObjects.cs
@@ -94,7 +94,7 @@ public ASRRecoveryServicesProvider()
         ///     required
         ///     parameters.
         /// </summary>
-        /// <param name="provider"></param>
+        /// <param name="provider">Recovery Service Provider object</param>
         public ASRRecoveryServicesProvider(
             RecoveryServicesProvider provider)
         {
@@ -132,7 +132,7 @@ public ASRRecoveryServicesProvider(
         public string ID { get; set; }
 
         /// <summary>
-        ///     Gets or sets the Type of Management entity ?VMM, V-Center.
+        ///     Gets or sets the Type of Management entity - VMM, V-Center.
         /// </summary>
         public string Type { get; set; }
 
@@ -181,6 +181,7 @@ public ASRFabric()
         /// <summary>
         ///     Initializes a new instance of the <see cref="ASRSite" /> class.
         /// </summary>
+        /// <param name="fabric">Fabric object</param>
         public ASRFabric(
             Fabric fabric)
         {
@@ -1445,6 +1446,7 @@ private void UpdateDiskDetails(
     public class ASRFabricSpecificVMDetails
     {
     }
+
     /// <summary>
     /// Azure Site Recovery Replication Protected Item.
     /// </summary>
@@ -2014,7 +2016,7 @@ public ASRGroupTaskDetails()
         /// <summary>
         ///     Initializes a new instance of the <see cref="ASRGroupTaskDetails" /> class.
         /// </summary>
-        /// <param name="groupTaskDetails"></param>
+        /// <param name="groupTaskDetails">Task details to load values from.</param>
         public ASRGroupTaskDetails(
             GroupTaskDetails groupTaskDetails)
         {
@@ -2743,9 +2745,6 @@ public ASRAzuretoAzureDiskReplicationConfig()
         {
         }
 
-        /// <summary>
-        ///     Initializes a new instance of the <see cref="ASRRunAsAccount" /> class.
-        /// </summary>
         /// <summary>
         /// Gets or sets the disk uri.
         /// </summary>
diff --git a/src/RecoveryServices/RecoveryServices.SiteRecovery/Utilities/Utilities.cs b/src/RecoveryServices/RecoveryServices.SiteRecovery/Utilities/Utilities.cs
index 91d279fac2f8..e14ba78de615 100644
--- a/src/RecoveryServices/RecoveryServices.SiteRecovery/Utilities/Utilities.cs
+++ b/src/RecoveryServices/RecoveryServices.SiteRecovery/Utilities/Utilities.cs
@@ -169,7 +169,7 @@ public static string GetProviderNameSpaceFromArmId(
         /// <summary>
         ///     Get Value from ARM ID
         /// </summary>
-        /// <param name="armId"></param>
+        /// <param name="armId">ARM Id.</param>
         /// <param name="key"></param>
         /// <returns>the key</returns>
         public static string GetValueFromArmId(
diff --git a/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesClient.cs b/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesClient.cs
index 4fb7b8a5bbbf..34bd496370cf 100644
--- a/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesClient.cs
+++ b/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesClient.cs
@@ -97,7 +97,7 @@ public ResourceManagementClient RmClient
         /// Initializes a new instance of the <see cref="PSRecoveryServicesClient" /> class with 
         /// required current subscription.
         /// </summary>
-        /// <param name="defaultContext"></param>
+        /// <param name="defaultContext">Azure context.</param>
         public PSRecoveryServicesClient(IAzureContext defaultContext)
         {
             string resourceType = string.Empty;
diff --git a/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesVaultClient.cs b/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesVaultClient.cs
index 9d0fec32d464..7bf70cbfca71 100644
--- a/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesVaultClient.cs
+++ b/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesVaultClient.cs
@@ -63,13 +63,13 @@ public Vault CreateVault(string resouceGroupName, string vaultName, Vault vault)
                 resouceGroupName, vaultName, vault, GetRequestHeaders()).Result.Body;
         }
 
-        /// <summary>  
+        /// <summary>
         /// Method to create or update Recovery Services Vault.
-        /// </summary>  
-        /// <param name="resouceGroupName">Name of the resouce group</param>  
-        /// <param name="vaultName">Name of the vault</param>  
+        /// </summary>
+        /// <param name="resouceGroupName">Name of the resouce group</param>
+        /// <param name="vaultName">Name of the vault</param>
         /// <param name="vault">patch vault object to patch the recovery services Vault</param>
-        /// <returns>Azure Recovery Services Vault.</returns> 
+        /// <returns>Azure Recovery Services Vault.</returns>
         public Vault UpdateRSVault(string resouceGroupName, string vaultName, PatchVault vault)
         {
             var response = GetRecoveryServicesClient.Vaults.UpdateWithHttpMessagesAsync(resouceGroupName, vaultName, vault).Result;
@@ -105,13 +105,13 @@ public Rest.Azure.AzureOperationResponse DeleteVault(string resouceGroupName, st
         }
 
 
-        /// <summary>  
-        /// Method to Update Azure Recovery Services Vault Backup Properties  
-        /// </summary>  
-        /// <param name="resouceGroupName">Name of the resouce group</param>  
+        /// <summary>
+        /// Method to Update Azure Recovery Services Vault Backup Properties
+        /// </summary>
+        /// <param name="resouceGroupName">Name of the resouce group</param>
         /// <param name="vaultName">Name of the vault</param>
-        /// <param name="backupStorageConfig"></param>  
-        /// <returns>Azure Operation response object.</returns>  
+        /// <param name="backupStorageConfig">Backup Properties Update</param>
+        /// <returns>Azure Operation response object.</returns>
         public void UpdateVaultStorageType(string resouceGroupName, string vaultName,
             BackupResourceConfigResource backupStorageConfig)
         {
@@ -119,13 +119,13 @@ public void UpdateVaultStorageType(string resouceGroupName, string vaultName,
                 vaultName, resouceGroupName, backupStorageConfig, GetRequestHeaders());
         }
 
-        /// <summary>  
-        /// Method to Patch Azure Recovery Services Vault Backup Properties  
-        /// </summary>  
-        /// <param name="resouceGroupName">Name of the resouce group</param>  
+        /// <summary>
+        /// Method to Patch Azure Recovery Services Vault Backup Properties
+        /// </summary>
+        /// <param name="resouceGroupName">Name of the resouce group</param>
         /// <param name="vaultName">Name of the vault</param>
-        /// <param name="backupStorageConfig"></param>  
-        /// <returns>Azure Operation response object.</returns>  
+        /// <param name="backupStorageConfig">Backup Properties Update</param>
+        /// <returns>Azure Operation response object.</returns>
         public void PatchVaultStorageConfigProperties(string resouceGroupName, string vaultName,
             BackupResourceConfigResource backupStorageConfig)
         {
@@ -133,12 +133,12 @@ public void PatchVaultStorageConfigProperties(string resouceGroupName, string va
                 vaultName, resouceGroupName, backupStorageConfig, GetRequestHeaders());
         }
 
-        /// <summary>  
-        /// Method to Get Azure Recovery Services Vault Backup Properties  
-        /// </summary>  
-        /// <param name="resouceGroupName">Name of the resouce group</param>  
-        /// <param name="vaultName">Name of the vault</param>  
-        /// <returns>Azure Resource Storage response object.</returns>  
+        /// <summary>
+        /// Method to Get Azure Recovery Services Vault Backup Properties
+        /// </summary>
+        /// <param name="resouceGroupName">Name of the resouce group</param>
+        /// <param name="vaultName">Name of the vault</param>
+        /// <returns>Azure Resource Storage response object.</returns>
         public BackupResourceConfigResource GetVaultStorageConfig(string resouceGroupName, string vaultName)
         {
             return GetRecoveryServicesBackupClient.BackupResourceStorageConfigsNonCRR.GetWithHttpMessagesAsync(
diff --git a/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesVaultExtendedInfoClient.cs b/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesVaultExtendedInfoClient.cs
index c53f5e4f3b35..555bdc47a6e2 100644
--- a/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesVaultExtendedInfoClient.cs
+++ b/src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesVaultExtendedInfoClient.cs
@@ -240,8 +240,8 @@ public ASRVaultCreds ChangeVaultContext(ARSVault vault)
         /// <summary>
         /// Changes the Vault context
         /// </summary>
-        /// <param name="vaultName">Name of the vault</param>
         /// <param name="resourceGroupName">Name of the resouce group</param>
+        /// <param name="vaultName">Name of the vault</param>
         /// <returns>credential object</returns>
         public long? getVaultAuthType(string resourceGroupName, string vaultName)
         {
diff --git a/src/RecoveryServices/RecoveryServices/Models/PSContracts.cs b/src/RecoveryServices/RecoveryServices/Models/PSContracts.cs
index 10b584b0fe61..8a484f2ae0ec 100644
--- a/src/RecoveryServices/RecoveryServices/Models/PSContracts.cs
+++ b/src/RecoveryServices/RecoveryServices/Models/PSContracts.cs
@@ -380,14 +380,14 @@ public ASRVaultCreds()
         /// <param name="subscriptionId">subscription Id</param>
         /// <param name="resourceName">resource name</param>
         /// <param name="managementCert">management cert</param>
-        /// <param name="acsNamespace">authenticating service  namespace</param>
+        /// <param name="acsNamespace">authenticating service namespace</param>
         /// <param name="channelIntegrityKey">Agent Channel Integrity Key</param>
         /// <param name="resourceGroupName">cloud service name</param>
         /// <param name="siteId">custom site Id</param>
         /// <param name="siteName">custom site name</param>
-        /// <param name="resourceNamespace">Name of the resouce group</param>
-        /// <param name="resourceType"></param>
-        /// <param name="location"></param>
+        /// <param name="resourceNamespace">resource namespace</param>
+        /// <param name="resourceType">resource type</param>
+        /// <param name="location">vault location</param>
         public ASRVaultCreds(
             string subscriptionId,
             string resourceName,
diff --git a/src/RecoveryServices/RecoveryServices/Utilities/Utilities.cs b/src/RecoveryServices/RecoveryServices/Utilities/Utilities.cs
index 2ddf41805dd1..2b5db78fd767 100644
--- a/src/RecoveryServices/RecoveryServices/Utilities/Utilities.cs
+++ b/src/RecoveryServices/RecoveryServices/Utilities/Utilities.cs
@@ -157,7 +157,7 @@ public static string GenerateRandomKey(int size)
         /// <summary>
         /// Get Value from ARM ID
         /// </summary>
-        /// <param name="armId"></param>
+        /// <param name="armId">ARM Id.</param>
         /// <param name="key"></param>
         /// <returns>the key</returns>
         public static string GetValueFromArmId(string armId, string key)
diff --git a/src/RecoveryServices/RecoveryServices/Vault/GetAzureRMRecoveryServicesVaultSettingsFile.cs b/src/RecoveryServices/RecoveryServices/Vault/GetAzureRMRecoveryServicesVaultSettingsFile.cs
index 372382356387..4564c174a66d 100644
--- a/src/RecoveryServices/RecoveryServices/Vault/GetAzureRMRecoveryServicesVaultSettingsFile.cs
+++ b/src/RecoveryServices/RecoveryServices/Vault/GetAzureRMRecoveryServicesVaultSettingsFile.cs
@@ -559,7 +559,7 @@ private string GenerateVaultCreds(X509Certificate2 cert, string subscriptionId,
         /// <summary>
         /// Generates vault creds file content for backup Vault
         /// </summary>
-        /// <param name="certificateString"></param>
+        /// <param name="certificateString">management certificate</param>
         /// <param name="subscriptionId">subscription Id</param>
         /// <param name="vaultCertificateResponse"></param>
         /// <returns>xml file in string format</returns>