Skip to content

Spelling Fixes #27960

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

Merged
merged 1 commit into from
Jun 14, 2025
Merged

Spelling Fixes #27960

merged 1 commit into from
Jun 14, 2025

Conversation

ArieHein
Copy link
Contributor

Description

Some more goodies @vidai-msft :)

This goes over all Az.Accounts tree and fixes all spelling mistakes in all codebase files that are either:

  • in comments.
  • in displayable strings.

As its all under the same cmdlet, i included all the fixes in one PR as i assumed it all goes through same testing cycle pre release of next version.

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

  • 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 13, 2025 21:47
Copy link

Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

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 multiple spelling mistakes across the Az.Accounts codebase by correcting typos in comments, XML documentation, and displayable strings without altering any programmatic behavior.

  • Fixed typos in XML documentation, inline comments, and exception messages.
  • Updated changelog entries and test comments to use correct terminology.
  • No functional changes introduced.

Reviewed Changes

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

Show a summary per file
File Description
src/Accounts/Accounts/CommonModule/UserAgent.cs Corrected a typo in the parameter description ("outgpoing" → "outgoing").
src/Accounts/Accounts/CommonModule/UniqueId.cs Corrected a typo in the parameter description ("outgpoing" → "outgoing").
src/Accounts/Accounts/CommonModule/TelemetryProvider.cs Fixed multiple typos in XML documentation comments and error message texts.
src/Accounts/Accounts/CommonModule/PSModuleExtensions.cs Corrected the misspelling in the summary comment ("Extnesion" → "Extension").
src/Accounts/Accounts/CommonModule/PSAzureServiceProfile.cs Corrected a typo in the summary comment ("descriptin" → "description").
src/Accounts/Accounts/CommonModule/IEventStore.cs Fixed a typo in the return description ("remainign" → "remaining").
src/Accounts/Accounts/CommonModule/Events.cs Corrected the event documentation spelling error ("validatio9n" → "validation").
src/Accounts/Accounts/CommonModule/EventHelper.cs Corrected typos within the method summaries ("extensaions" → "extensions", "containign" → "containing", "cotnainng" → "containing").
src/Accounts/Accounts/CommonModule/EventData.cs Fixed several typos in the remarks and code example comments.
src/Accounts/Accounts/CommonModule/EnvironmentExtensions.cs Corrected multiple spelling issues in the XML documentation (e.g. "naseUri" → "baseUri", "Determien" → "Determine", "contaisn" → "contains", "siffix" → "suffix").
src/Accounts/Accounts/CommonModule/AzModule.cs Updated the comment to fix the spelling error ("telemtry" → "telemetry").
src/Accounts/Accounts/ChangeLog.md Updated changelog entries to fix typos (e.g. "influeced" → "influenced", "buildin" → "builtin").
src/Accounts/Accounts/AutoSave/DisableAzureRmContextAutosave.cs Corrected the inline comment spelling ("explicitely" → "explicitly").
src/Accounts/Accounts/Account/ConnectAzureRmAccount.cs Fixed typos in parameter HelpMessage and inline comments ("certficate" → "certificate", "visiable" → "visible", "telemtry" → "telemetry").
src/Accounts/Accounts.Test/SubscriptionCmdletTests.ps1 Fixed a typo in a test comment ("subscripiton" → "subscription").
src/Accounts/Accounts.Test/Mocks/MockSubscriptionClientFactoryVersion2019.cs Corrected the error message typo ("Subscripiton" → "Subscription").
src/Accounts/Accounts.Test/Mocks/MockDataStore.cs Fixed a typo in the comment ("Regeular" → "Regular").
src/Accounts/Accounts.Test/DefaultCmdletTests.ps1 Corrected the test comment from "Resoure" to "Resource".
src/Accounts/Accounts.Test/ConnectAzAccountScenarioTests.ps1 Fixed typos in comments ("freamwork" → "framework").
src/Accounts/Accounts.Test/AzureRMProfileTests.cs Updated error messages in CloudException to fix typos ("tenenat" → "tenant").

Copy link
Contributor

Thank you for your contribution @ArieHein! We will review the pull request and get back to you soon.

@vidai-msft
Copy link
Contributor

Much appreciated @ArieHein for all the contributions to our repo. I feel the majority of the major and critical spelling issues have been addressed especially after this PR is merged for Accounts module. Please feel free to explore other areas that you are interested as I understand the work is a bit tedious. Thank you again for your support and effort. :-)

@vidai-msft
Copy link
Contributor

/azp run azure-powershell - security-tools

Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@vidai-msft vidai-msft enabled auto-merge (squash) June 14, 2025 13:56
@vidai-msft vidai-msft merged commit 72e3acf into Azure:main Jun 14, 2025
13 checks passed
@ArieHein ArieHein deleted the wave-01 branch June 14, 2025 17:13
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