Skip to content

Spelling Fixes #28050

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 1 commit into
base: main
Choose a base branch
from
Open

Spelling Fixes #28050

wants to merge 1 commit into from

Conversation

ArieHein
Copy link
Contributor

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

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

@Copilot Copilot AI review requested due to automatic review settings June 29, 2025 22:39
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 @ArieHein! We will review the pull request and get back to you soon.

Copy link
Contributor

@Copilot Copilot AI left a 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 numerous typos in comments, error messages, examples, and documentation across various files in the codebase.

  • Correct spelling mistakes in comments and documentation strings.
  • Update error messages and example outputs to reflect proper spelling.
  • Adjust resource strings and changelogs for consistent terminology.

Reviewed Changes

Copilot reviewed 23 out of 26 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/ElasticSan/ElasticSan.Autorest/test/Remove-AzElasticSanVolumeGroupNetworkRule.Tests.ps1 Fixed typo in comment (“exsiting” to “existing”).
src/ElasticSan/ElasticSan.Autorest/test/Add-AzElasticSanVolumeGroupNetworkRule.Tests.ps1 Fixed typo in comment (“exsiting” to “existing”).
src/ElasticSan/ElasticSan.Autorest/README.md Fixed typo in comment (“paramaters” to “parameters”).
src/Elastic/Elastic.Autorest/examples/Get-AzElasticVersion.md Corrected output text (“Lattest” to “Latest”).
src/Elastic/Elastic.Autorest/docs/Get-AzElasticVersion.md Corrected output text (“Lattest” to “Latest”).
src/EdgeOrder/EdgeOrder.Autorest/test/Update-AzEdgeOrderItem.Tests.ps1 Corrected parameter name (“ShippingAddres” to “ShippingAddress”).
src/EdgeOrder/EdgeOrder.Autorest/test/Update-AzEdgeOrderAddress.Tests.ps1 Corrected parameter name (“ShippingAddres” to “ShippingAddress”).
src/EdgeOrder/EdgeOrder.Autorest/examples/Get-AzEdgeOrderItem.md Corrected typo in variable (“ordderItem” to “orderItem”).
src/EdgeOrder/EdgeOrder.Autorest/docs/Get-AzEdgeOrderItem.md Corrected typo in variable (“ordderItem” to “orderItem”).
src/DynatraceObservability/DynatraceObservability.Autorest/examples/Get-AzDynatraceMonitorVMHostPayload.md Fixed typo (“coammnd” to “command”).
src/DynatraceObservability/DynatraceObservability.Autorest/examples/Get-AzDynatraceMonitor.md Fixed typo (“subsciption” to “subscription”) in title and description.
src/DynatraceObservability/DynatraceObservability.Autorest/docs/Get-AzDynatraceMonitorVMHostPayload.md Fixed typo (“coammnd” to “command”).
src/DynatraceObservability/DynatraceObservability.Autorest/docs/Get-AzDynatraceMonitor.md Fixed typo (“subsciption” to “subscription”) in title and description.
src/DynatraceObservability/DynatraceObservability.Autorest/README.md Fixed typo (“varinat” to “variant”).
src/DnsResolver/DnsResolver.Autorest/test/utils.ps1 Fixed typo (“Provison” to “Provision”) in comment.
src/DnsResolver/DnsResolver.Autorest/test/Get-AzDnsForwardingRuleset.Tests.ps1 Fixed typo (“resouce” to “resource”) in description.
src/Dns/Dns/Models/DnsRecordSet.cs Fixed typo (“temrinating” to “terminating”) in XML comment.
src/DeviceProvisioningServices/DeviceProvisioningServices/Properties/Resources.resx Corrected spelling errors (“Provisioing” to “Provisioning”) in resource strings.
src/DeviceProvisioningServices/DeviceProvisioningServices/Models/PSIotDpsSkuInfo.cs Fixed typo (“provisoning” to “provisioning”).
src/DeviceProvisioningServices/DeviceProvisioningServices/Models/PSIotDpsPropertiesDescription.cs Fixed typos (“provisoning” to “provisioning” and “assosciated” to “associated”).
src/DeviceProvisioningServices/DeviceProvisioningServices/Cmdlet/IotDpsDataPlane/Enrollment/AddAzIotDeviceProvisioningServiceEnrollment.cs Fixed typo (“requried” to “required”) in exception message.
src/DeviceProvisioningServices/DeviceProvisioningServices/ChangeLog.md Fixed typo (“exsting” to “existing”) in changelog entry.
src/DeviceProvisioningServices/DeviceProvisioningServices.Test/ScenarioTests/IotDpsEnrollmentTests.ps1 Fixed typo (“requried” to “required”) in test error message.
Files not reviewed (1)
  • src/DeviceProvisioningServices/DeviceProvisioningServices/Properties/Resources.Designer.cs: Language not supported
Comments suppressed due to low confidence (29)

src/ElasticSan/ElasticSan.Autorest/test/Remove-AzElasticSanVolumeGroupNetworkRule.Tests.ps1:18

  • Typo detected: 'exsiting' should be 'existing' in the comment.
    It 'NetworkRuleResourceId' {

src/ElasticSan/ElasticSan.Autorest/test/Remove-AzElasticSanVolumeGroupNetworkRule.Tests.ps1:32

  • Typo detected: 'exsiting' should be 'existing' in the comment.
        # Clear existing rules 

src/ElasticSan/ElasticSan.Autorest/test/Add-AzElasticSanVolumeGroupNetworkRule.Tests.ps1:19

  • Typo detected: 'exsiting' should be 'existing' in the comment.
        # Clear existing rules 

src/ElasticSan/ElasticSan.Autorest/test/Add-AzElasticSanVolumeGroupNetworkRule.Tests.ps1:29

  • Typo detected: 'exsiting' should be 'existing' in the comment.
        # Clear existing rules 

src/ElasticSan/ElasticSan.Autorest/README.md:62

  • Typo detected: 'paramaters' should be 'parameters' in the comment.
  # Set alias for Name parameters

src/Elastic/Elastic.Autorest/examples/Get-AzElasticVersion.md:8

  • Typo detected: 'Lattest' should be 'Latest' in the output example.
---------

src/Elastic/Elastic.Autorest/docs/Get-AzElasticVersion.md:33

  • Typo detected: 'Lattest' should be 'Latest' in the output example.
8.15.2 (Latest)

src/EdgeOrder/EdgeOrder.Autorest/test/Update-AzEdgeOrderItem.Tests.ps1:20

  • Typo detected: 'ShippingAddres' should be 'ShippingAddress' in the parameter name.
        $ShippingDetails = New-AzEdgeOrderShippingAddressObject -StreetAddress1 $env.StreetAddress1 -StateOrProvince $env.StateOrProvince -Country $env.Country -City $env.City -PostalCode $env.PostalCode -AddressType $env.AddressType 

src/EdgeOrder/EdgeOrder.Autorest/test/Update-AzEdgeOrderAddress.Tests.ps1:20

  • Typo detected: 'ShippingAddres' should be 'ShippingAddress' in the parameter name.
        $ShippingDetails = New-AzEdgeOrderShippingAddressObject -StreetAddress1 $env.StreetAddress1 -StateOrProvince $env.StateOrProvince -Country $env.Country -City $env.City -PostalCode $env.PostalCode -AddressType $env.AddressType 

src/EdgeOrder/EdgeOrder.Autorest/examples/Get-AzEdgeOrderItem.md:4

  • Typo detected: 'ordderItem' should be 'orderItem' in the example command.
$orderItem | Format-List

src/EdgeOrder/EdgeOrder.Autorest/docs/Get-AzEdgeOrderItem.md:41

  • Typo detected: 'ordderItem' should be 'orderItem' in the example command.
$orderItem | Format-List

src/DynatraceObservability/DynatraceObservability.Autorest/examples/Get-AzDynatraceMonitorVMHostPayload.md:12

  • Typo detected: 'coammnd' should be 'command' in the documentation.
This command gets the payload that needs to be passed in the request body for installing Dynatrace agent on a VM.

src/DynatraceObservability/DynatraceObservability.Autorest/examples/Get-AzDynatraceMonitor.md:1

  • Typo detected: 'subsciption' should be 'subscription'.
### Example 1: List all dynatrace monitors under a subscription

src/DynatraceObservability/DynatraceObservability.Autorest/docs/Get-AzDynatraceMonitorVMHostPayload.md:36

  • Typo detected: 'coammnd' should be 'command' in the documentation.
This command gets the payload that needs to be passed in the request body for installing Dynatrace agent on a VM.

src/DynatraceObservability/DynatraceObservability.Autorest/docs/Get-AzDynatraceMonitor.md:43

  • Typo detected: 'subsciption' should be 'subscription'.
### Example 1: List all dynatrace monitors under a subscription

src/DynatraceObservability/DynatraceObservability.Autorest/docs/Get-AzDynatraceMonitor.md:54

  • Typo detected: 'subsciption' should be 'subscription'.
This command lists all dynatrace monitors under a subscription.

src/DynatraceObservability/DynatraceObservability.Autorest/README.md:89

  • Typo detected: 'varinat' should be 'variant'.
  # remove variant

src/DnsResolver/DnsResolver.Autorest/test/utils.ps1:73

  • Typo detected: 'Provison' should be 'Provision' in the comment.
    # Provision of virtual network and generating DNS Resolver names.

src/DnsResolver/DnsResolver.Autorest/test/Get-AzDnsForwardingRuleset.Tests.ps1:55

  • Typo detected: 'resouce' should be 'resource' in the test description.
    It 'List all DNS forwarding ruleset under the resource group, expect all DNS forwarding rulesets retrieved' {

src/Dns/Dns/Models/DnsRecordSet.cs:630

  • Typo detected: 'temrinating' should be 'terminating' in the XML comment.
        /// <summary>

src/DeviceProvisioningServices/DeviceProvisioningServices/Properties/Resources.resx:123

  • Typo detected: 'Provisioing' should be 'Provisioning' in the resource string.
  <data name="AddCertificate" xml:space="preserve">

src/DeviceProvisioningServices/DeviceProvisioningServices/Properties/Resources.resx:138

  • Typo detected: 'Provisioing' should be 'Provisioning' in the resource string.
  <data name="NewCertificateVerificationCode" xml:space="preserve">

src/DeviceProvisioningServices/DeviceProvisioningServices/Models/PSIotDpsSkuInfo.cs:19

  • Typo detected: 'provisoning' should be 'provisioning' in the comment.
    /// <summary>

src/DeviceProvisioningServices/DeviceProvisioningServices/Models/PSIotDpsPropertiesDescription.cs:20

  • Typo detected: 'provisoning' should be 'provisioning' in the comment.
    /// <summary>

src/DeviceProvisioningServices/DeviceProvisioningServices/Models/PSIotDpsPropertiesDescription.cs:44

  • Typo detected: 'assosciated' should be 'associated' in the comment.
        /// <summary>

src/DeviceProvisioningServices/DeviceProvisioningServices/Cmdlet/IotDpsDataPlane/Enrollment/AddAzIotDeviceProvisioningServiceEnrollment.cs:202

  • Typo detected: 'requried' should be 'required' in the exception message.
                        {

src/DeviceProvisioningServices/DeviceProvisioningServices/ChangeLog.md:36

  • Typo detected: 'exsting' should be 'existing' in the changelog entry.
* Allowed update of attestation information (e.g. keys, certificates) for existing device enrollments and enrollment groups

src/DeviceProvisioningServices/DeviceProvisioningServices.Test/ScenarioTests/IotDpsEnrollmentTests.ps1:80

  • Typo detected: 'requried' should be 'required' in the test error message.
	# Expecting error while creating device enrollment with TPM attestation mechanism

@YanaXu
Copy link
Contributor

YanaXu commented Jun 30, 2025

/azp run

Copy link
Contributor

Azure Pipelines successfully started running 3 pipeline(s).

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.

2 participants