-
Notifications
You must be signed in to change notification settings - Fork 4k
fix spelling #27893
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
base: main
Are you sure you want to change the base?
fix spelling #27893
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 batches spelling and minor formatting fixes across various PowerShell help markdown files to improve documentation accuracy.
- Corrected typos in parameter descriptions and example commands
- Fixed duplicated text in a code example
- Standardized naming and spacing in help content
Reviewed Changes
Copilot reviewed 21 out of 21 changed files in this pull request and generated 3 comments.
Show a summary per file
File | Description |
---|---|
src/App/App/help/Get-AzContainerAppManagedEnvCert.md | Removed duplicated command in Example 3 |
src/ApiManagement/ApiManagement/help/Update-AzApiManagementRegion.md | Corrected “ResoureId” to “ResourceId” |
src/ApiManagement/ApiManagement/help/Remove-AzApiManagementUser.md | Fixed typo “$Ture” to “$True” |
src/ApiManagement/ApiManagement/help/New-AzApiManagementRegion.md | Corrected “ResoureId” to “ResourceId” |
src/ApiManagement/ApiManagement/help/New-AzApiManagementKeyVaultObject.md | Fixed extra “e” in “PsApiManagementKeyVaultObjecte” → “PsApiManagementKeyVaultObject” |
src/ApiManagement/ApiManagement/help/New-AzApiManagementGatewayHostnameConfiguration.md | Fixed “configuratin”/“confiuration” → “configuration” |
src/ApiManagement/ApiManagement/help/New-AzApiManagement.md | Corrected “ResoureId” to “ResourceId” |
src/ApiManagement/ApiManagement/help/Az.ApiManagement.md | Fixed “configuratin” → “configuration” |
src/ApiManagement/ApiManagement/help/Add-AzApiManagementRegion.md | Corrected “ResoureId” to “ResourceId” |
src/AlertsManagement/AlertsManagement/help/Update-AzSmartGroupState.md | Fixed “Acknowleged” → “Acknowledged” |
src/AlertsManagement/AlertsManagement/help/Set-AzAlertProcessingRule.md | Corrected “Reccurent” → “Recurrent” |
src/AlertsManagement/AlertsManagement/help/Measure-AzAlertStatistic.md | Corrected “Moniter” → “Monitor” |
src/Aks/Aks/help/Update-AzAksNodePool.md | Fixed “minimun” → “minimum” |
src/Aks/Aks/help/Set-AzAksCluster.md | Corrected “enalbe” → “enable” and “lenth” → “length” |
src/Aks/Aks/help/New-AzAksNodePool.md | Fixed “myResouceGroup” → “myResourceGroup” |
src/Aks/Aks/help/New-AzAksCluster.md | Corrected “enalbe” → “enable” |
src/Aks/Aks/help/Install-AzAksCliTool.md | Fixed “lateset” → “latestversion” |
src/Accounts/Accounts/help/Connect-AzAccount.md | Corrected “CertficatePath” → “CertificatePath” and “certficate” → “certificate” |
src/Accounts/Accounts/help/About/about_az.md | Fixed “MyResoureGroup” → “MyResourceGroup” |
src/AlertsManagement/AlertsManagement/help/Set-AzAlertProcessingRule.md
Outdated
Show resolved
Hide resolved
src/AlertsManagement/AlertsManagement/help/Set-AzAlertProcessingRule.md
Outdated
Show resolved
Hide resolved
adding additional findings Co-authored-by: Copilot <[email protected]>
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.
Thanks for the contribution. Could you please help remove the unnecessary spaces as commented?
@@ -435,7 +435,7 @@ Accept wildcard characters: False | |||
|
|||
### -ScheduleReccurence2EndTime | |||
Reccurence Start Time in the timezone of ScheduleTimeZone parameter. Format 06:00:00 | |||
Should be mentioned in case of Reccurent Schedule - Daily, Weekly or Monthly. | |||
Should be mentioned in case of Recurrent Schedule - Daily, Weekly or Monthly. |
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.
One more space between Recurrent and Schedule. Could you please remove it?
@@ -451,7 +451,7 @@ Accept wildcard characters: False | |||
|
|||
### -ScheduleReccurence2StartTime | |||
Reccurence Start Time in the timezone of ScheduleTimeZone parameter. Format 06:00:00 | |||
Should be mentioned in case of Reccurent Schedule - Daily, Weekly or Monthly. | |||
Should be mentioned in case of Recurrent Schedule - Daily, Weekly or Monthly. |
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.
One more space between Recurrent and Schedule. Could you please remove it?
@@ -515,7 +515,7 @@ Accept wildcard characters: False | |||
|
|||
### -ScheduleReccurenceEndTime | |||
Reccurence Start Time in the timezone of ScheduleTimeZone parameter. Format 06:00:00 | |||
Should be mentioned in case of Reccurent Schedule - Daily, Weekly or Monthly. | |||
Should be mentioned in case of Recurrent Schedule - Daily, Weekly or Monthly. |
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.
One more space between Recurrent and Schedule. Could you please remove it?
@@ -531,7 +531,7 @@ Accept wildcard characters: False | |||
|
|||
### -ScheduleReccurenceStartTime | |||
Reccurence Start Time in the timezone of ScheduleTimeZone parameter. Format 06:00:00 | |||
Should be mentioned in case of Reccurent Schedule - Daily, Weekly or Monthly. | |||
Should be mentioned in case of Recurrent Schedule - Daily, Weekly or Monthly. |
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.
One more space between Recurrent and Schedule. Could you please remove it?
This PR was labeled "needs-revision" because it has unresolved review comments or CI failures. |
Description
As the documentations is moving to this repo and will be generated, im adding quite a few pull requests with spelling fixes.
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.