-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Spelling Fixes #27960
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 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"). |
Thank you for your contribution @ArieHein! We will review the pull request and get back to you soon. |
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. :-) |
/azp run azure-powershell - security-tools |
Azure Pipelines successfully started running 1 pipeline(s). |
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:
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
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.