-
Notifications
You must be signed in to change notification settings - Fork 4k
Spelling Fixes #27963
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 #27963
Conversation
Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status. |
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 fixes various spelling mistakes in comments, display strings, and documentation examples across multiple files. The changes improve clarity in user-facing messages and inline comments without affecting code behavior.
- Corrected typos such as "subscrition" ➔ "subscription", "endpint" ➔ "endpoint", "disbale" ➔ "disable", and several other spelling errors.
- Updated PowerShell script comments and markdown examples to ensure consistency in wording.
Reviewed Changes
Copilot reviewed 35 out of 35 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
src/Cdn/Cdn.Autorest/docs/Test-AzFrontDoorCdnProfileMigration.md | Corrected "subscrition" to "subscription" in examples |
src/Cdn/Cdn.Autorest/docs/Test-AzCdnNameAvailability.md | Fixed "endpint" to "endpoint" in title and description |
src/Cdn/Cdn.Autorest/docs/Stop-AzFrontDoorCdnProfileMigration.md | Fixed "subscrition" to "subscription" in examples and comments |
src/Cdn/Cdn.Autorest/docs/Start-AzFrontDoorCdnProfilePrepareMigration.md | Corrected various instances of "subscrition" to "subscription" |
src/Cdn/Cdn.Autorest/docs/New-AzFrontDoorCdnProfileScrubbingRulesObject.md | Updated "disbale" to "disable" in examples |
src/Cdn/Cdn.Autorest/docs/Invoke-AzCdnCommitProfileToAFDMigration.md | Corrected "subscrition" to "subscription" in example descriptions |
src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnOriginGroupResourceUsage.md | Fixed "useages" to "usages" in example titles and descriptions |
src/Cdn/Cdn.Autorest/docs/Get-AzFrontDoorCdnEndpointResourceUsage.md | Fixed "useages" to "usages" in example titles and descriptions |
src/Cdn/Cdn.Autorest/docs/Get-AzCdnEndpointResourceUsage.md | Fixed "useages" to "usages" in example titles and descriptions |
src/Cdn/Cdn.Autorest/docs/Enable-AzFrontDoorCdnProfileMigration.md | Corrected "subscrition" to "subscription" in the description |
src/Cdn/Cdn.Autorest/custom/Update-AzFrontDoorCdnProfileSku.ps1 | Fixed typos in comments ("subscritpion", "subsrciption" ➔ "subscription") |
src/Cdn/Cdn.Autorest/custom/Start-AzFrontDoorCdnProfilePrepareMigration.ps1 | Corrected spelling mistakes in comments ("consistence" ➔ "consistence", "subsrciption" ➔ "subscription", etc.) |
src/Cdn/Cdn.Autorest/README.md | Fixed typos such as "cmelet" ➔ "cmdlet" and "inconvinient" ➔ "inconvenient"; adjusted "ojects" ➔ "objects" |
src/BotService/LiveTests/BotService.Autorest/TestLiveScenarios.disabled.ps1 | Fixed typo "servcie" ➔ "service" in test scenario name |
src/Blueprint/Blueprint/Models/PSBlueprintAssignment.cs | Corrected "PSBluprintAssignment" comment to "PSBlueprintAssignment" |
src/Blueprint/Blueprint/Common/Utils.cs | Fixed "subcscription" to "subscription" in a comment |
src/BillingBenefits/BillingBenefits.Autorest/examples/Invoke-AzBillingBenefitsElevateSavingPlanOrder.md | Corrected "identiy" to "identity" in example title |
src/BillingBenefits/BillingBenefits.Autorest/docs/Invoke-AzBillingBenefitsElevateSavingPlanOrder.md | Corrected "identiy" to "identity" in example title |
src/Billing/UsageAggregates/TestTheCmdLet.ps1 | Fixed variable naming typo ("agggregate" ➔ "aggregate") in script and output messages |
src/Billing/Consumption.Test/ScenarioTests/BudgetTests.ps1 | Corrected typo "notificaiton" ➔ "notification" in debug message |
/azp run azure-powershell - security-tools |
Azure Pipelines successfully started running 1 pipeline(s). |
src/Cdn/Cdn.Autorest/custom/Start-AzFrontDoorCdnProfilePrepareMigration.ps1
Show resolved
Hide resolved
src/Cdn/Cdn.Autorest/test/Test-AzFrontDoorCdnEndpointCustomDomain.Recording.json
Outdated
Show resolved
Hide resolved
src/Cdn/Cdn.Autorest/test/Test-AzFrontDoorCdnEndpointCustomDomain.Recording.json
Outdated
Show resolved
Hide resolved
@vidai-msft on the same topic of generation, there are some files under the Generated subfolder in each module, if there are spelling fixes there, where is the original source of them ? |
@ArieHein The source is the swagger repo. https://github.com/Azure/azure-rest-api-specs |
/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:
One change to a ps1 file for testing that is scoped to this file only.
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.