From da5762a9bb6496c9bc75e326c783652819d23ac8 Mon Sep 17 00:00:00 2001
From: Ziyue Zheng <v-ziyzhe@microsoft.com>
Date: Wed, 22 Jun 2022 14:24:55 +0800
Subject: [PATCH] Fix Az.StorageSync XML comments (#18603)

---
 .../Converters/BackgroundDataDownloadActivityConverter.cs     | 2 +-
 .../StorageSync/Common/StorageSyncClientWrapper.cs            | 4 +---
 .../StorageSync/Common/StorageSyncResourceManager.cs          | 2 --
 .../Interop/Clients/SyncServerCertificateRolloverClient.cs    | 2 --
 src/StorageSync/StorageSync/Models/PSRegisteredServer.cs      | 2 +-
 .../StorageSync/ServerEndpoint/NewServerEndpointCommand.cs    | 4 ++--
 6 files changed, 5 insertions(+), 11 deletions(-)

diff --git a/src/StorageSync/StorageSync/Common/Converters/BackgroundDataDownloadActivityConverter.cs b/src/StorageSync/StorageSync/Common/Converters/BackgroundDataDownloadActivityConverter.cs
index 05d6ba75116f..0057fde8c3e9 100644
--- a/src/StorageSync/StorageSync/Common/Converters/BackgroundDataDownloadActivityConverter.cs
+++ b/src/StorageSync/StorageSync/Common/Converters/BackgroundDataDownloadActivityConverter.cs
@@ -20,7 +20,7 @@ namespace Microsoft.Azure.Commands.StorageSync.Common.Converters
 
     /// <summary>
     /// Class BackgroundDataDownloadActivityConverter.
-    /// Implements the <see cref="Converters.ConverterBase{PSBackgroundDataDownloadActivity, ServerEndpointBackgroundDataDownloadActivity}"
+    /// Implements the <see cref="Converters.ConverterBase{PSBackgroundDataDownloadActivity, ServerEndpointBackgroundDataDownloadActivity}" />
     /// </summary>
     /// <seealso cref="Converters.ConverterBase{PSBackgroundDataDownloadActivity, ServerEndpointBackgroundDataDownloadActivity}" />
     public class BackgroundDataDownloadActivityConverter : ConverterBase<PSBackgroundDataDownloadActivity, StorageSyncModels.ServerEndpointBackgroundDataDownloadActivity>
diff --git a/src/StorageSync/StorageSync/Common/StorageSyncClientWrapper.cs b/src/StorageSync/StorageSync/Common/StorageSyncClientWrapper.cs
index 733897f7e217..9b0491805dfc 100644
--- a/src/StorageSync/StorageSync/Common/StorageSyncClientWrapper.cs
+++ b/src/StorageSync/StorageSync/Common/StorageSyncClientWrapper.cs
@@ -38,11 +38,9 @@ namespace Microsoft.Azure.Commands.StorageSync.Common
 {
     /// <summary>
     /// Class StorageSyncClientWrapper.
-    /// Implements the <see cref="Microsoft.Azure.Commands.StorageSync.Common.IStorageSyncClientWrapper" />
     /// Implements the <see cref="Microsoft.Azure.Commands.StorageSync.Interfaces.IStorageSyncClientWrapper" />
     /// </summary>
     /// <seealso cref="Microsoft.Azure.Commands.StorageSync.Interfaces.IStorageSyncClientWrapper" />
-    /// <seealso cref="Microsoft.Azure.Commands.StorageSync.Common.IStorageSyncClientWrapper" />
     public class StorageSyncClientWrapper : IStorageSyncClientWrapper
     {
         /// <summary>
@@ -108,7 +106,7 @@ public class StorageSyncClientWrapper : IStorageSyncClientWrapper
         /// Initializes a new instance of the <see cref="StorageSyncClientWrapper" /> class.
         /// </summary>
         /// <param name="context">The context.</param>
-        /// <param name="activeDirectoryClient">The active directory client.</param>
+        /// <param name="microsoftGraphClient">The active directory client.</param>
         public StorageSyncClientWrapper(IAzureContext context, MicrosoftGraphClient microsoftGraphClient)
                 : this(AzureSession.Instance.ClientFactory.CreateArmClient<StorageSyncManagementClient>(context, AzureEnvironment.Endpoint.ResourceManager),
                       AzureSession.Instance.ClientFactory.CreateArmClient<AuthorizationManagementClient>(context, AzureEnvironment.Endpoint.ResourceManager),
diff --git a/src/StorageSync/StorageSync/Common/StorageSyncResourceManager.cs b/src/StorageSync/StorageSync/Common/StorageSyncResourceManager.cs
index 86ad4733f8ff..1d90a2b48014 100644
--- a/src/StorageSync/StorageSync/Common/StorageSyncResourceManager.cs
+++ b/src/StorageSync/StorageSync/Common/StorageSyncResourceManager.cs
@@ -24,11 +24,9 @@ namespace Microsoft.Azure.Commands.StorageSync.Common
 {
     /// <summary>
     /// Class StorageSyncResourceManager.
-    /// Implements the <see cref="Microsoft.Azure.Commands.StorageSync.Common.IStorageSyncResourceManager" />
     /// Implements the <see cref="Microsoft.Azure.Commands.StorageSync.Interfaces.IStorageSyncResourceManager" />
     /// </summary>
     /// <seealso cref="Microsoft.Azure.Commands.StorageSync.Interfaces.IStorageSyncResourceManager" />
-    /// <seealso cref="Microsoft.Azure.Commands.StorageSync.Common.IStorageSyncResourceManager" />
     public class StorageSyncResourceManager : IStorageSyncResourceManager
     {
 
diff --git a/src/StorageSync/StorageSync/Interop/Clients/SyncServerCertificateRolloverClient.cs b/src/StorageSync/StorageSync/Interop/Clients/SyncServerCertificateRolloverClient.cs
index c8ec178d70e7..858e09e63e27 100644
--- a/src/StorageSync/StorageSync/Interop/Clients/SyncServerCertificateRolloverClient.cs
+++ b/src/StorageSync/StorageSync/Interop/Clients/SyncServerCertificateRolloverClient.cs
@@ -24,11 +24,9 @@ namespace Commands.StorageSync.Interop.Clients
 {
     /// <summary>
     /// Function performs server certificate rollover
-    /// Implements the <see cref="StorageSync.Management.PowerShell.Cmdlets.CertificateRollover.ISyncServerCertificateRollover" />
     /// Implements the <see cref="Commands.StorageSync.Interop.Interfaces.ISyncServerCertificateRollover" />
     /// </summary>
     /// <seealso cref="Commands.StorageSync.Interop.Interfaces.ISyncServerCertificateRollover" />
-    /// <seealso cref="StorageSync.Management.PowerShell.Cmdlets.CertificateRollover.ISyncServerCertificateRollover" />
     public class SyncServerCertificateRolloverClient : ISyncServerCertificateRollover
     {
         /// <summary>
diff --git a/src/StorageSync/StorageSync/Models/PSRegisteredServer.cs b/src/StorageSync/StorageSync/Models/PSRegisteredServer.cs
index 7bba60a99eed..c9e90b1d8168 100644
--- a/src/StorageSync/StorageSync/Models/PSRegisteredServer.cs
+++ b/src/StorageSync/StorageSync/Models/PSRegisteredServer.cs
@@ -149,7 +149,7 @@ public class PSRegisteredServer : PSResourceBase
         /// <value>The management endpoint URI.</value>
         public string ManagementEndpointUri { get; set; }
 
-        // <summary>
+        /// <summary>
         /// Gets or sets the monitoring endpoint URI.
         /// </summary>
         /// <value>The monitoring endpoint URI.</value>
diff --git a/src/StorageSync/StorageSync/ServerEndpoint/NewServerEndpointCommand.cs b/src/StorageSync/StorageSync/ServerEndpoint/NewServerEndpointCommand.cs
index 5a366b68464b..be4227e9b7e8 100644
--- a/src/StorageSync/StorageSync/ServerEndpoint/NewServerEndpointCommand.cs
+++ b/src/StorageSync/StorageSync/ServerEndpoint/NewServerEndpointCommand.cs
@@ -170,7 +170,7 @@ public class NewServerEndpointCommand : StorageSyncClientCmdletBase
           HelpMessage = HelpMessages.TierFilesOlderThanDaysParameter)]
         public int? TierFilesOlderThanDays { get; set; }
 
-        // <summary>
+        /// <summary>
         /// Gets or sets a value indicating the policy to use for the initial download sync.
         /// </summary>
         /// <value>The initial download policy.</value>
@@ -197,7 +197,7 @@ public class NewServerEndpointCommand : StorageSyncClientCmdletBase
             IgnoreCase = true)]
         public string LocalCacheMode { get; set; }
 
-        // <summary>
+        /// <summary>
         /// Gets or sets a value indicating the policy to use for the initial upload sync.
         /// </summary>
         /// <value>The initial upload policy.</value>