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

Fix spell check to detect Spanish spelling errors #5007

Closed
4 tasks done
chalin opened this issue Aug 9, 2024 · 4 comments
Closed
4 tasks done

Fix spell check to detect Spanish spelling errors #5007

chalin opened this issue Aug 9, 2024 · 4 comments
Assignees
Labels
bug Something isn't working CI/infra CI & infrastructure

Comments

@chalin
Copy link
Contributor

chalin commented Aug 9, 2024

Follow up to:

Spelling check failed, but was not reported as a failure :-/ ... see https://github.com/open-telemetry/opentelemetry.io/actions/runs/10317103275/job/28560650923

Run streetsidesoftware/cspell-action@v6
cspell-action
Pull Request
Error: Configuration
        name: Configuration Loader Error
        msg: Failed to resolve configuration file: "@cspell/dict-es-es/cspell-ext.json" referenced from "./.cspell.yml"

Here are the current errors:

$ npm run check:spelling                          

> check:spelling
> npx cspell --no-progress -c .cspell.yml content data 'layouts/**/*.md'

content/es/_index.md:4:3 - Unknown word (Telemetría)
content/es/_index.md:4:59 - Unknown word (observabilidad)
content/es/_index.md:48:54 - Unknown word (telemetría)
content/es/_index.md:62:28 - Unknown word (telemetría)
content/es/_index.md:63:1 - Unknown word (reenvíalos)
content/es/_index.md:80:60 - Unknown word (observabilidad)
CSpell: Files checked: 1312, Issues found: 6 in 1 file.

Tasks

/cc @theletterf

@chalin chalin added bug Something isn't working CI/infra CI & infrastructure labels Aug 9, 2024
@chalin
Copy link
Contributor Author

chalin commented Aug 9, 2024

@theletterf - I'm willing to fix the workflow issues. I'll let you address the spelling errors.

@theletterf
Copy link
Member

Oh whoops. Going to fix that!

@chalin
Copy link
Contributor Author

chalin commented Aug 9, 2024

Oh, this has been failing even since we added pt-br spell-check support! Upstream issue:

@chalin
Copy link
Contributor Author

chalin commented Aug 9, 2024

Closed by #5008 and #5009

@chalin chalin closed this as completed Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CI/infra CI & infrastructure
Projects
Status: Done
Development

No branches or pull requests

2 participants