Skip to content

Fix typo 'DeyAll' to 'DenyAll' in NetworkAccessPolicy description #28394

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Ramlah7
Copy link

@Ramlah7 Ramlah7 commented Aug 14, 2025

Description

Fixed a typo in the NewAzureRmDiskUpdateConfigCommand.cs parameter description for New-AzDiskUpdateConfig where DeyAll was written instead of DenyAll.
Updated the source comment so the generated documentation will be correct in future builds.

Closes #28210

Mandatory Checklist

  • Please choose the target release of Azure PowerShell.

    • No need for a release
  • Check this box to confirm: I have read the Submitting Changes section of CONTRIBUTING.md and reviewed the following information:

  • SHOULD update ChangeLog.md file(s) appropriately
    • No ChangeLog.md update required for this documentation typo fix.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
    • No cmdlet API change — only fixed text in doc comment.
  • SHOULD have proper test coverage for changes in pull request.
    • Not applicable — documentation-only change.
  • SHOULD NOT adjust version of module manually in pull request

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 @Ramlah7! We will review the pull request and get back to you soon.

@Ramlah7 Ramlah7 mentioned this pull request Aug 14, 2025
@NoriZC NoriZC added the Compute label Aug 15, 2025
@NoriZC
Copy link
Contributor

NoriZC commented Aug 15, 2025

/azp run

Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

NoriZC
NoriZC previously approved these changes Aug 15, 2025
@Ramlah7
Copy link
Author

Ramlah7 commented Aug 15, 2025

@microsoft-github-policy-service agree

@Ramlah7 Ramlah7 requested a review from NoriZC August 15, 2025 23:41
@VeryEarly
Copy link
Collaborator

Copy link

This PR was labeled "needs-revision" because it has unresolved review comments or CI failures.
Please resolve all open review comments and make sure all CI checks are green. Refer to our guide to troubleshoot common CI failures.

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.

[Doc]: Typo in doc
3 participants