-
Notifications
You must be signed in to change notification settings - Fork 4k
Spelling Fixes #28070
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Spelling Fixes #28070
Conversation
Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
Thank you for your contribution @ArieHein! We will review the pull request and get back to you soon. |
There was a problem hiding this 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. |
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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> |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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. |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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.
// 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.
/azp run azure-powershell - security-tools |
Azure Pipelines successfully started running 1 pipeline(s). |
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
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.md
and reviewed the following information:ChangeLog.md
file(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
.## Upcoming Release
header in the past tense.ChangeLog.md
if no new release is required, such as fixing test case only.