Skip to content
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

translation/azure-ai-translation-document - cspell found spelling errors in source code #22684

Closed
danieljurek opened this issue Jan 21, 2022 · 0 comments · Fixed by #22858
Closed
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Translator

Comments

@danieljurek
Copy link
Member

danieljurek commented Jan 21, 2022

Spell check scanning of package at sdk/translation/azure-ai-translation-document detected spelling errors in the source code and related files. This directory is opted out of PR spell checking in PR #22199 to keep PRs unblocked.

What to do

  1. Ensure Node.js is installed (https://nodejs.org/en/download/).
  2. Delete the entry in .vscode/cspell.json's ignorePaths field. It will look like: sdk/translation/azure-ai-translation-document/**. You need to do this to enable checking the files.
  3. From the root of the repo run spell check using ./eng/common/spelling/Invoke-Cspell.ps1 -ScanGlobs "sdk/translation/azure-ai-translation-document/**"
  4. Fix detections according to http://aka.ms/azsdk/engsys/spellcheck use the "False positives" section to fix false positives
  5. Check in changes (including the change to .vscode/cspell.json where the ignorePaths is updated to remove the entry for this service.). You may need to run git add -f .vscode/cspell.json to force adding the changes to the file in git.

Spell checking output

./sdk/translation/azure-ai-translation-document/azure/ai/translation/document/_client.py:255:61 - Unknown word (descendingly)
./sdk/translation/azure-ai-translation-document/azure/ai/translation/document/_client.py:327:50 - Unknown word (descendingly)
./sdk/translation/azure-ai-translation-document/azure/ai/translation/document/_helpers.py:21:29 - Unknown word (Apim)
./sdk/translation/azure-ai-translation-document/azure/ai/translation/document/_helpers.py:112:14 - Unknown word (apim)
./sdk/translation/azure-ai-translation-document/azure/ai/translation/document/aio/_client_async.py:267:61 - Unknown word (descendingly)
./sdk/translation/azure-ai-translation-document/azure/ai/translation/document/aio/_client_async.py:336:50 - Unknown word (descendingly)
./sdk/translation/azure-ai-translation-document/samples/assets/glossary_sample.csv:1:10 - Unknown word (crâne)
./sdk/translation/azure-ai-translation-document/samples/assets/glossary_sample.csv:3:7 - Unknown word (cœur)
./sdk/translation/azure-ai-translation-document/samples/assets/glossary_sample.csv:4:7 - Unknown word (poumons)
./sdk/translation/azure-ai-translation-document/samples/assets/glossary_sample.tsv:1:12 - Unknown word (crâne)
./sdk/translation/azure-ai-translation-document/samples/assets/glossary_sample.tsv:3:9 - Unknown word (cœur)
./sdk/translation/azure-ai-translation-document/samples/assets/glossary_sample.tsv:4:9 - Unknown word (poumons)
./sdk/translation/azure-ai-translation-document/samples/assets/glossary_sample.xlf:7:39 - Unknown word (crâne)
./sdk/translation/azure-ai-translation-document/samples/assets/glossary_sample.xlf:15:36 - Unknown word (cœur)
./sdk/translation/azure-ai-translation-document/samples/assets/glossary_sample.xlf:19:36 - Unknown word (poumons)
./sdk/translation/azure-ai-translation-document/samples/async_samples/sample_translation_with_azure_blob_async.py:33:15 - Unknown word (mydocument)
./sdk/translation/azure-ai-translation-document/samples/sample_translation_with_azure_blob.py:33:15 - Unknown word (mydocument)
./sdk/translation/azure-ai-translation-document/tests/asynctestcase.py:81:21 - Unknown word (parms)
./sdk/translation/azure-ai-translation-document/tests/glossaries-valid.csv:1:9 - Unknown word (essai)
./sdk/translation/azure-ai-translation-document/tests/glossaries-valid.csv:2:6 - Unknown word (textes)
./sdk/translation/azure-ai-translation-document/tests/test_list_document_statuses_async.py:158:9 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_document_statuses_async.py:162:64 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_document_statuses_async.py:163:13 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_document_statuses_async.py:180:9 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_document_statuses.py:136:9 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_document_statuses.py:138:64 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_document_statuses.py:139:13 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_document_statuses.py:155:9 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations_async.py:8:8 - Unknown word (pytz)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations_async.py:167:48 - Unknown word (pytz)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations_async.py:194:9 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations_async.py:196:67 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations_async.py:197:13 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations_async.py:214:9 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations_async.py:231:50 - Unknown word (pytz)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations_async.py:233:48 - Unknown word (pytz)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations.py:8:8 - Unknown word (pytz)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations.py:155:48 - Unknown word (pytz)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations.py:182:9 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations.py:184:67 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations.py:185:13 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations.py:202:9 - Unknown word (curr)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations.py:219:50 - Unknown word (pytz)
./sdk/translation/azure-ai-translation-document/tests/test_list_translations.py:221:48 - Unknown word (pytz)
./sdk/translation/azure-ai-translation-document/tests/test_translation_async.py:553:18 - Unknown word (essai)
./sdk/translation/azure-ai-translation-document/tests/test_translation.py:554:18 - Unknown word (essai)
./sdk/translation/azure-ai-translation-document/tests/testcase.py:65:60 - Unknown word (Apim)
./sdk/translation/azure-ai-translation-document/tests/testcase.py:173:9 - Unknown word (inprogress)
./sdk/translation/azure-ai-translation-document/tests/testcase.py:173:34 - Unknown word (inprogress)
./sdk/translation/azure-ai-translation-document/tests/testcase.py:186:74 - Unknown word (inprogress)
./sdk/translation/azure-ai-translation-document/tests/testcase.py:186:89 - Unknown word (inprogress)
./sdk/translation/azure-ai-translation-document/tests/testcase.py:285:21 - Unknown word (parms)
@danieljurek danieljurek added the Client This issue points to a problem in the data-plane of the library. label Jan 21, 2022
@danieljurek danieljurek changed the title translation/azure-ai-translation-document - cspell found spelling errors in public API surface translation/azure-ai-translation-document - cspell found spelling errors in source code Jan 21, 2022
@kristapratico kristapratico self-assigned this Jan 21, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Apr 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. Cognitive - Translator
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants