Spelling Fixes #27961
Merged
Spelling Fixes #27961
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
There was a problem hiding this comment.
Pull Request Overview
This pull request addresses various spelling corrections throughout the codebase including comments, display strings, XML documentation, and test scripts. The key changes are:
- Correcting spelling mistakes in user-facing messages and comments.
- Fixing typos in parameter help messages and documentation.
- Updating a PowerShell cmdlet name typo in test scripts.
Reviewed Changes
Copilot reviewed 30 out of 31 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/ApiManagement/ApiManagement.Test/ScenarioTests/ApiManagementTests.ps1 | Fixed typo in comment ("idenity" → "identity") and updated cmdlet name in tests. |
| src/ApiManagement/ApiManagement.ServiceManagement/Models/PsApiManagementApiVersionSet.cs | Corrected spelling of "identifer" to "identifier" in XML documentation. |
| src/ApiManagement/ApiManagement.ServiceManagement/Commands/RemoveAzureApiManagementPolicy.cs | Corrected "requied" to "required" in help message. |
| src/ApiManagement/ApiManagement.ServiceManagement/Commands/NewAzureApiManagementUser.cs | Fixed typo "genetated" to "generated" in help message. |
| src/ApiManagement/ApiManagement.ServiceManagement/Commands/NewAzureApiManagementGatewayHostnameConfiguration.cs | Fixed typo "confiuration" to "configuration" in help message. |
| src/ApiManagement/ApiManagement.ServiceManagement.Test/ScenarioTests/ApiManagementTests.ps1 | Corrected cmdlet name typo from "Subscripiton" to "Subscription". |
| src/ApiManagement/ApiManagement.ServiceManagement.Test/Resources/petstoreOpenApi.json | Fixed typo "Muliple" to "Multiple" in API description. |
| src/AnalysisServices/AnalysisServices/Commands/GetAzureRmAnalysisServicesServer.cs | Corrected misspelling "avaliable" to "available" in comment. |
| src/AnalysisServices/AnalysisServices.Dataplane/Models/AsAzureDataplaneClient.cs | Fixed "Asynchronosly" to "Asynchronously" and "cancelation" to "cancellation" in comments. |
| src/AlertsManagement/AlertsManagement/ChangeLog.md | Corrected service name in changelog to "Alert". |
| src/AlertsManagement/AlertsManagement/AlertCommands/MeasureAzureAlertStatistic.cs | Fixed typo "Moniter" to "Monitor" in help messages. |
| src/AlertsManagement/AlertsManagement/ActionRuleCommands/SetAzureAlertProcessingRule.cs | Updated various occurrences of "Reccurent/Reccurence/Reccurence" to "Recurrent/Recurrence" in help messages and comments. |
| src/Aks/Aks/Properties/Resources.resx | Fixed several typos (“craete”, “paramters”, “Reseting”, “anothe”) in resource strings. |
| src/Aks/Aks/Models/PSKubernetesCluster.cs | Corrected "FDQN" to "FQDN" in XML documentation. |
| src/Aks/Aks/Models/PSContainerServiceAgentPoolProfile.cs | Corrected "FDQN" to "FQDN" in XML documentation. |
| src/Aks/Aks/Commands/SetAzureRmAks.cs | Fixed typo "enalbe" to "enable" in help message. |
| src/Aks/Aks/Commands/NewAzureRmAks.cs | Fixed typo "enalbe" to "enable" in help message. |
| src/Aks/Aks/Commands/CreateOrUpdateKubeBase.cs | Fixed typo "lenth" to "length" in help message. |
| src/Aks/Aks.Test/ScenarioTests/NodePoolTests.ps1 | Corrected typo "creat" to "create" in comment. |
Files not reviewed (1)
- src/Aks/Aks/Properties/Resources.Designer.cs: Language not supported
|
Thank you for your contribution @ArieHein! We will review the pull request and get back to you soon. |
|
/azp run azure-powershell - security-tools |
|
Azure Pipelines successfully started running 1 pipeline(s). |
vidai-msft
approved these changes
Jun 14, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This goes spelling fixes in codebase files that are either:
and one occasion of bad PowerShell cmdlet in the tests.
There are NO changes to programmatic aspects or code behavior.
Im not sure you need to create a release of this, but i assume the next time this module code will be built, it will also include the spelling fixes.
If otherwise, please advise and ill add missing files, though im not able to do a full build / unit tests for example.
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.mdand reviewed the following information:ChangeLog.mdfile(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.## Upcoming Releaseheader in the past tense.ChangeLog.mdif no new release is required, such as fixing test case only.