resolved copilot comments

Author: hiaga

src/RecoveryServices/RecoveryServices.Backup/Cmdlets/Item/RedoAzureRmRecoveryServicesBackupProtection.cs

@@ -66,8 +66,7 @@ public override void ExecuteCmdlet()
                 base.ExecuteCmdlet();
                 PsBackupProviderManager providerManager;
                 JobBase jobObj = null;
-                                
-                // chck with nandini, source vault is default and can we name targetvault ? 
+                
                 ResourceIdentifier resourceIdentifier = new ResourceIdentifier(VaultId);
                 string vaultName = resourceIdentifier.ResourceName;
                 string resourceGroupName = resourceIdentifier.ResourceGroupName;
@@ -149,7 +148,7 @@ public override void ExecuteCmdlet()
 
                 // Step 3: Configure backup in target vault
                 WriteVerbose("Configuring backup in target vault now");
-                // chck : switch context to the target vault's subscription
+                // TODO : switch context to the target vault's subscription
 
                 // Create provider manager for target vault with workload-specific parameters
                 Dictionary<Enum, object> targetProviderParams = new Dictionary<Enum, object>()
@@ -190,7 +189,7 @@ public override void ExecuteCmdlet()
                 {
                     // For AzureFiles: extract file share name and storage account name
                     AzureFileShareItem afsItem = (AzureFileShareItem)Item;
-                    string fileShareName = afsItem.FriendlyName; // do we need name here?
+                    string fileShareName = afsItem.FriendlyName;
                     string storageAccountName = BackupUtils.GetStorageAccountNameFromContainerName(afsItem.ContainerName);
 
                     targetProviderParams.Add(ItemParams.ItemName, fileShareName);

src/RecoveryServices/RecoveryServices.Backup/Helpers/CmdletHelper.cs

@@ -42,16 +42,16 @@ public static bool IsContainerUnregistrationRequired(CmdletModel.ContainerType c
 
         public static string GetContainerNameFromItem(ItemBase item)
         {
+            // container name format for SQL and AFS
+            // StorageContainer;Storage;rgname;afsname
+            // VMAppContainer;Compute;RGName;vmanme
             return item.ContainerName;
         }
 
         public static JobBase UnregisterContainer(ItemBase item, string vaultName, string resourceGroupName, ServiceClientAdapter serviceClientAdapter, RSBackupVaultCmdletBase rsBackupVaultCmdletBase)
         {
             string containerName = GetContainerNameFromItem(item);
-            // chck: verify if this is the correct format for SQL and AFS workloads
-            // StorageContainer;Storage;rgname;afsname
-            // VMAppContainer;Compute;RGName;vmanme
-
+            
             Logger.Instance.WriteDebug("Unregistering Azure Storage container: " + containerName);
             try
             {
@@ -158,7 +158,7 @@ public static ProtectableItemBase GetMatchingProtectableItem(ItemBase item, stri
                         Logger.Instance.WriteDebug("proItemName: " + ((AzureWorkloadProtectableItem)itemModel).Name +
                             ", proItemContainerName: " + ((AzureWorkloadProtectableItem)itemModel).ContainerName);
 
-                        return (string.Compare(((AzureWorkloadProtectableItem)itemModel).Name, item.Name,true) == 0
+                        return (string.Compare(((AzureWorkloadProtectableItem)itemModel).Name, item.Name, true) == 0
                         && string.Compare(((AzureWorkloadProtectableItem)itemModel).ContainerName, item.ContainerName, true) == 0);
                     }).ToList();
                 }
@@ -168,7 +168,7 @@ public static ProtectableItemBase GetMatchingProtectableItem(ItemBase item, stri
                     return null;
                 }
 
-                if(itemModels!= null && itemModels.Count > 0)
+                if(itemModels != null && itemModels.Count > 0)
                 {
                     return itemModels[0];
                 }