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.

Sign up for GitHub

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

Jump to bottom

Spell check action produces many false positives #8678

Closed
kmk3 opened this issue May 16, 2021 · 1 comment
Closed

Spell check action produces many false positives #8678

kmk3 opened this issue May 16, 2021 · 1 comment
Labels
frozen-due-to-age

Comments

@kmk3
Copy link
Contributor

kmk3 commented May 16, 2021

From #8676 (comment):

@github-actions:

@check-spelling-bot Report

Unrecognized words, please review:

  • Alexey
  • Anatoliy
  • andreyvit
  • Ankit
  • Anning
  • awood
  • bellvat
  • benbalter
  • Beney
  • Bou
  • Chayoung
  • Checkoway
  • Dassonville
  • Dousse
  • Dusseau
  • eol
  • Espinaco
  • Finazzo
  • Gaudino
  • githubcom
  • Goulven
  • Hoizey
  • jan
  • Jekylling
  • Kacper
  • Kasberg
  • Kewin
  • Kinnula
  • Kolesky
  • Kotvinsky
  • Kulig
  • Kwokfu
  • localheinz
  • Locher
  • Manmeet
  • Maroli
  • Marsceill
  • Maximiliano
  • mentoring
  • Mertcan
  • mertkahyaoglu
  • Mittal
  • mmistakes
  • myproject
  • Nadjib
  • nakanishi
  • nior
  • Pelykh
  • Porcel
  • Ruparelia
  • Rusiczki
  • Schroers
  • shingo
  • Singhaniya
  • Taillandier
  • Theunissen
  • Tyborska
  • Vasovi
  • Vishesh
  • vohedge
  • wildcards
  • Xhmikos
  • Xiaoiver
  • Yashu
  • Yastreb
  • Zlatan
Log excerpt from the workflow run of #8676[1] 

Checking spelling...
(...Spell check files...)
Checking 108 files
(...Spell check...)
Warning: docs/_posts/2014-12-17-alfredxing-welcome-to-jekyll-core.md: line 23, columns 7-16, Warning - `Jekylling` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-03-10-making-it-easier-to-contribute-to-jekyll.md: line 4, columns 9-18, Warning - `benbalter` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-03-10-making-it-easier-to-contribute-to-jekyll.md: line 12, columns 379-388, Warning - `githubcom` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 37, columns 65-74, Warning - `myproject` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 95, columns 52-58, Warning - `Maroli` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 95, columns 10-17, Warning - `Yastreb` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 95, columns 1-9, Warning - `Anatoliy` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 95, columns 27-34, Warning - `Gaudino` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 96, columns 21-28, Warning - `Finazzo` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 97, columns 63-74, Warning - `Taillandier` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 97, columns 31-39, Warning - `Espinaco` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 97, columns 1-4, Warning - `Bou` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 98, columns 48-55, Warning - `Kolesky` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 98, columns 15-21, Warning - `Kwokfu` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 99, columns 35-39, Warning - `nior` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 99, columns 63-70, Warning - `Manmeet` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 99, columns 23-30, Warning - `Kinnula` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 100, columns 34-41, Warning - `Mertcan` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 100, columns 26-32, Warning - `Locher` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 101, columns 10-16, Warning - `Nadjib` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 101, columns 1-8, Warning - `Kasberg` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 101, columns 31-37, Warning - `Hoizey` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 101, columns 47-53, Warning - `Porcel` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 102, columns 16-25, Warning - `Marsceill` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 102, columns 69-75, Warning - `Zlatan` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 102, columns 35-44, Warning - `Checkoway` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 103, columns 1-7, Warning - `Vasovi` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 103, columns 26-32, Warning - `shingo` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 103, columns 48-55, Warning - `vohedge` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 103, columns 33-42, Warning - `nakanishi` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 103, columns 11-24, Warning - `mertkahyaoglu` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 13, columns 171-180, Warning - `andreyvit` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 13, columns 140-145, Warning - `awood` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 15, columns 235-244, Warning - `mmistakes` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 189-197, Warning - `Schroers` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 336-342, Warning - `Anning` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 268-275, Warning - `Goulven` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 81-89, Warning - `Tyborska` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 295-303, Warning - `Rusiczki` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 447-458, Warning - `Maximiliano` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 170-175, Warning - `Beney` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 37-43, Warning - `Alexey` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 539-548, Warning - `Ruparelia` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 44-50, Warning - `Pelykh` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 256-266, Warning - `Theunissen` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 91-96, Warning - `Ankit` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 524-529, Warning - `Kulig` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 124-131, Warning - `bellvat` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 252-255, Warning - `jan` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 391-397, Warning - `Kacper` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 550-558, Warning - `Xiaoiver` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 59-70, Warning - `Dassonville` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 531-538, Warning - `Vishesh` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 459-468, Warning - `Kotvinsky` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 97-107, Warning - `Singhaniya` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 569-575, Warning - `Mittal` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 141-148, Warning - `Dusseau` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 563-568, Warning - `Yashu` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 420-425, Warning - `Kewin` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 426-432, Warning - `Dousse` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-25-jekyll-3-7-2-released.md: line 25, columns 53-62, Warning - `wildcards` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-25-jekyll-3-7-2-released.md: line 40, columns 15-25, Warning - `localheinz` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-25-jekyll-3-7-2-released.md: line 61, columns 43-51, Warning - `Chayoung` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-03-14-development-update.md: line 10, columns 75-78, Warning - `EOL` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-03-14-development-update.md: line 24, columns 211-220, Warning - `mentoring` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 43, columns 42-51, Warning - `myproject` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 63, columns 62-71, Warning - `myproject` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2016-10-06-jekyll-3-3-is-here.md: line 109, columns 7-16, Warning - `Jekylling` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 15, columns 289-298, Warning - `mmistakes` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 116-122, Warning - `Maroli` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 234-245, Warning - `Taillandier` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-02-jekyll-3-7-0-released.md: line 23, columns 156-163, Warning - `Finazzo` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-25-jekyll-3-7-2-released.md: line 30, columns 26-35, Warning - `mmistakes` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-25-jekyll-3-7-2-released.md: line 61, columns 35-41, Warning - `Maroli` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-25-jekyll-3-7-2-released.md: line 62, columns 7-18, Warning - `Taillandier` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-25-jekyll-3-7-2-released.md: line 62, columns 39-45, Warning - `Kacper` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-01-25-jekyll-3-7-2-released.md: line 67, columns 7-16, Warning - `Jekylling` is not a recognized word. (unrecognized-spelling)
Warning: docs/_posts/2018-03-14-development-update.md: line 26, columns 7-16, Warning - `Jekylling` is not a recognized word. (unrecognized-spelling)
(...Compare expect with new output...)
(...New output...)
(...Unrecognized...)
Preparing a comment for pull_request_target
https://api.github.com/repos/jekyll/jekyll/issues/8676/comments
Error: Process completed with exit code 1.

Most of these seem to be names of contributors.

And these words are already correct:

  • eol
  • wildcards

This issue was likely caused by #8675.

Suggestion: Ignore the docs/_posts directory to avoid having to start
adding/maintaining contributor names on the dictionary.

[1] https://github.com/jekyll/jekyll/pull/8676/checks?check_run_id=2594957351

Cc: @jsoref (as the author of #8675)
@kmk3 kmk3 changed the title Spelling action produces many false positives Spell check action produces many false positives May 16, 2021
@kmk3 kmk3 mentioned this issue May 16, 2021
Merged
@kmk3
Copy link
Contributor Author

kmk3 commented May 16, 2021

Fixed by #8677.

@kmk3 kmk3 closed this as completed May 16, 2021
@jekyll jekyll locked and limited conversation to collaborators May 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
frozen-due-to-age
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jekyllbot @kmk3