search

Azure / azure-sdk-for-net

This repository is for active development of the Azure SDK for .NET. For consumers of the SDK we recommend visiting our public developer docs at https://learn.microsoft.com/dotnet/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-net.
MIT License
5.5k stars 4.82k forks source link

Fix spelling errors in code #17798

Closed danieljurek closed 8 months ago

danieljurek commented 3 years ago

Follow this process to eliminate spelling errors: https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/406/Spell-Checking

See ignorePaths in cspell.json (when checked in https://github.com/Azure/azure-sdk-for-net/pull/17762) to find files which contain spelling errors. Files that are in ignorePaths will not be checked for spelling errors until they are removed from ignorePaths.

Spelling errors may be as innocuous as misspelled words in comments, false positives (words that are correctly spelled but not recognized by the spell checker), or more severe resulting in interface changes (e.g. changing the name of a class or method name). Spelling errors should be examined and fixed by an appropriate SME.

This work item might be larger than a single issue. It might make sense to convert this to an epic and create issues for different areas of ownership (like per-service or per-package).

kurtzeborn commented 3 years ago

This issue is tracking the fixing of spelling errors in the code, so flipping this back to "Client". The EngSys team has added spell checking to our pipelines such that PR validation should start failing if spelling errors are introduced. In order to do this we had to add all existing spelling failures to an exception list. This issue is tracking fixing all of the exceptions.

ghost commented 3 years ago

Hi @danieljurek. There hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by removing the no-recent-activity label. Otherwise, we'll close this out in 7 days.

maririos commented 3 years ago
maririos commented 3 years ago

Documentation: https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/406/Spell-Checking

maririos commented 3 years ago

Services that have errors/warnings:

Status Service Issue
agrifood #23172
communication #23173
compute #23174
containerregistry #23000, #23175
cosmosdb #23176
digitaltwins #23177
dns #23178
eventgrid #23002, #23179
✔️ eventhub #23003
✔️ formrecognizer
insights #23180
✔️ keyvault #23004
machinelearningservices
modelsrepository #23182
monitor #23183
network #23184
objectanchors #23186
purviewpurview #23185
remoterendering #23187
✔️ search #23005
✔️ servicebus #23006
✔️ storage
sqlmanagement #23188
synapse #23007, #23190
✔️ textanalytics #22916
✔️ translation #22917
videoanalyzer #23189
danieljurek commented 3 years ago

Updated table with current status from https://github.com/Azure/azure-sdk-for-net/pull/23150

JimSuplizio commented 8 months ago

Hi @danieljurek, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.