Skip to content

Conversation

ArieHein
Copy link
Contributor

Description

This does spelling fixes in codebase files that are either:

in comments.
in displayable strings.
There are NO other changes to programmatic aspects or code behavior.

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

@Copilot Copilot AI review requested due to automatic review settings June 30, 2025 19:33
Copy link

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

Copy link
Contributor

Thank you for your contribution @ArieHein! We will review the pull request and get back to you soon.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR addresses various spelling mistakes in comments and displayable strings across the codebase. The changes fix typos such as "resouce" to "resource", "auduence" to "audience", "excution" to "execution", and many similar corrections.

Reviewed Changes

Copilot reviewed 27 out of 28 changed files in this pull request and generated 44 comments.

Show a summary per file
File Description
src/RecoveryServices/RecoveryServices/Vault/GetAzureRmRecoveryServicesVaults.cs Fixed spelling errors in XML comments describing resource groups.
src/RecoveryServices/RecoveryServices/Vault/GetAzureRMRecoveryServicesVaultSettingsFile.cs Corrected typo in return description ("auduence" to "audience").
src/RecoveryServices/RecoveryServices/Models/PSParameterSets.cs Fixed typo in comment ("excution" to "execution").
src/RecoveryServices/RecoveryServices/Models/PSObjects.cs Corrected spelling of "Resource" in comment.
src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesVaultExtendedInfoClient.cs Fixed typos in comments ("improvve" to "improve" and "resouce" to "resource").
src/RecoveryServices/RecoveryServices/Common/PSRecoveryServicesVaultClient.cs Multiple spelling fixes for "resouceGroup" and similar terms in XML documentation.
src/RecoveryServices/RecoveryServices/ChangeLog.md Corrected typos in changelog entries.
src/RecoveryServices/RecoveryServices.Backup/RecoveryServicesBackupCmdletBase.cs Fixed typo in exception logging comment ("occuring" to "occurring").
src/RecoveryServices/RecoveryServices.Backup/ParamHelpMsgs.cs Fixed typos in help messages (e.g. "operaiton" to "operation", "corss" to "cross").
src/RecoveryServices/RecoveryServices.Backup/Cmdlets/Vault/TestAzureRMRecoveryServicesDSMove.cs Corrected typo ("validtaing" to "validating").
src/RecoveryServices/RecoveryServices.Backup/Cmdlets/Vault/SetAzureRmRecoveryServicesVaultProperty.cs Fixed comment typos ("chck" to "check").
src/RecoveryServices/RecoveryServices.Backup/Cmdlets/Restore/GetAzureRmRecoveryServicesBackupWorkloadRecoveryConfig.cs Corrected typo in comment ("TragetItem" to "TargetItem").
src/RecoveryServices/RecoveryServices.Backup/Cmdlets/ProtectionPolicy/NewAzureRmRecoveryServicesBackupProtectionPolicy.cs Fixed typo ("assoicated" to "associated").
src/RecoveryServices/RecoveryServices.Backup/Cmdlets/Item/GetAzureRmRecoveryServicesBackupItem.cs Corrected typo ("Bakup" to "Backup").
src/RecoveryServices/RecoveryServices.Backup/Cmdlets/Container/UndoAzureRmRecoveryServicesBackupContainerDeletion.cs Fixed typo in comment ("contianer" to "container").
src/RecoveryServices/RecoveryServices.Backup/Cmdlets/Container/RegisterAzureRmRecoveryServicesBackupContainer.cs Corrected typo in comment ("contiane" to "container").
src/RecoveryServices/RecoveryServices.Backup/Cmdlets/Container/GetAzureRmRecoveryServicesBackupContainer.cs Corrected typo in comment ("NotRegitered" to "NotRegistered").
src/RecoveryServices/RecoveryServices.Backup.Models/Properties/Resources.resx Fixed spelling mistakes in resource strings.
src/RecoveryServices/RecoveryServices.Backup.Models/CommonModels/PolicyScheduleObjects.cs Corrected typo ("greator" to "greater").
src/RecoveryServices/RecoveryServices.Backup.Models/CommonModels/PolicyRetentionObjects.cs Fixed multiple occurrences of the typo "rentention" to "retention".
src/RecoveryServices/RecoveryServices.Backup.Models/BaseObjects.cs Fixed typo ("rentention" to "retention") in base class comment.
src/RecoveryServices/RecoveryServices.Backup.Models/AzureVmWorkloadModels/AzureWorkloadRecoveryPoint.cs Corrected typo ("rediness" to "readiness").
src/RecoveryServices/RecoveryServices.Backup.Models/AzureVmModels/AzureVmRecoveryPoint.cs Corrected typo ("rediness" to "readiness").
src/RecoveryServices/RecoveryServices.Backup.Helpers/Validations/PolicyValidations.cs Fixed spelling mistakes ("rentention" to "retention") in validation comments.
src/RecoveryServices/RecoveryServices.Backup.Helpers/HelperUtils.cs Corrected typo ("polcy" to "policy") in XML comment.
src/RecoveryServices/RecoveryServices.Backup.Helpers/Conversions/RecoveryPointConversions.cs Fixed typo ("readness" to "readiness") in a comment.
src/RecoveryServices/RecoveryServices.Backup.Helpers/Conversions/ConversionHelpers.cs Corrected typo ("paased" to "passed") in a comment.
Files not reviewed (1)
  • src/RecoveryServices/RecoveryServices.Backup.Models/Properties/Resources.Designer.cs: Language not supported

@@ -102,7 +102,7 @@ public override void ExecuteCmdlet()
}

/// <summary>
/// Get vaults under a resouce group.
/// Get vaults under a resource group.
Copy link
Preview

Copilot AI Jun 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected 'resouce' to 'resource' in the XML comment.

Copilot uses AI. Check for mistakes.

@@ -113,7 +113,7 @@ private void GetVaultsUnderResourceGroup(string resourceGroupName)
}

/// <summary>
/// Get vaults under all resouce group.
/// Get vaults under all resource groups.
Copy link
Preview

Copilot AI Jun 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected 'resouce' to 'resource' in the XML comment.

Copilot uses AI. Check for mistakes.

@@ -681,7 +681,7 @@ private string GenerateVaultCredsForSiteRecovery(string subscriptionId,
/// <param name="vaultLocation">Vault location</param>
/// <param name="vaultName">Vault name</param>
/// <param name="resourceId">Resource Id</param>
/// <returns>AAD auduence</returns>
/// <returns>AAD audience</returns>
Copy link
Preview

Copilot AI Jun 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected 'auduence' to 'audience' in the XML documentation.

Copilot uses AI. Check for mistakes.

@@ -17,7 +17,7 @@ internal static class ARSParameterSets
internal const string ByDefault = "ByDefault";

/// <summary>
/// When excution has to be done for site
/// When execution has to be done for site
Copy link
Preview

Copilot AI Jun 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected 'excution' to 'execution' in the comment.

Copilot uses AI. Check for mistakes.

@@ -213,7 +213,7 @@ public ARSVault(Vault vault)
public string Location { get; set; }

/// <summary>
/// Gets or sets Resouce group name.
/// Gets or sets Resource group name.
Copy link
Preview

Copilot AI Jun 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected 'Resouce' to 'Resource' in the comment.

Copilot uses AI. Check for mistakes.

@@ -129,7 +129,7 @@ public static void ValidateFullBackupRetentionPolicyWithTieringPolicy(LongTermRe
}

/// <summary>
/// Helper function to validate long term rentention policy and simple schedule policy.
/// Helper function to validate long term retention policy and simple schedule policy.
Copy link
Preview

Copilot AI Jun 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected 'rentention' to 'retention' in the comment.

Copilot uses AI. Check for mistakes.

@@ -226,7 +226,7 @@ public static void ValidateLongTermRetentionPolicyWithSimpleSchedulePolicy(
}

/// <summary>
/// Helper function to validate long term rentention policy and simple schedule policy.
/// Helper function to validate long term retention policy and simple schedule policy.
Copy link
Preview

Copilot AI Jun 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected 'rentention' to 'retention' in the comment.

Copilot uses AI. Check for mistakes.

@@ -214,7 +214,7 @@ string id
}

/// <summary>
/// Gets polcy name from the provided dictionary of key value pairs.
/// Gets policy name from the provided dictionary of key value pairs.
Copy link
Preview

Copilot AI Jun 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected 'polcy' to 'policy' in the comment.

Copilot uses AI. Check for mistakes.

@@ -82,7 +82,7 @@ public static List<RecoveryPointBase> FilterRPsBasedOnTier(List<RecoveryPointBas
}

/// <summary>
/// filter move readness based on target tier
/// filter move readiness based on target tier
Copy link
Preview

Copilot AI Jun 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected 'readness' to 'readiness' in the comment.

Copilot uses AI. Check for mistakes.

@@ -886,7 +886,7 @@ public static void GetPSSubProtectionPolicy(AzureVmWorkloadPolicy azureVmWorkloa
}
else if (string.Compare(subProtectionPolicy.PolicyType, "Log") == 0)
{
// timeZone paased as input but not used in below method calls
// timeZone passed as input but not used in below method calls
Copy link
Preview

Copilot AI Jun 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corrected 'paased' to 'passed' in the comment.

Suggested change
// timeZone passed as input but not used in below method calls
// timeZone passed as input; verify its usage in the methods below

Copilot uses AI. Check for mistakes.

@YanaXu
Copy link
Contributor

YanaXu commented Jul 1, 2025

/azp run azure-powershell - security-tools

Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@YanaXu YanaXu enabled auto-merge (squash) July 1, 2025 02:12
@YanaXu YanaXu merged commit 3727599 into Azure:main Jul 1, 2025
13 checks passed
@ArieHein ArieHein deleted the wave-25 branch July 1, 2025 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants