Skip to content

[web] Improve check spelling settings#465

Merged
dgdavid merged 5 commits intomasterfrom
improve-cspell-config
Mar 14, 2023
Merged

[web] Improve check spelling settings#465
dgdavid merged 5 commits intomasterfrom
improve-cspell-config

Conversation

@dgdavid
Copy link
Contributor

@dgdavid dgdavid commented Mar 14, 2023

Problem

Thanks to @imobachgs's review at #462 (comment), I realize that something was not working as expected with the check spelling task introduced in #448

Basically, using the allowCompoundWords: true goes to let go some typos.

Solution

  • Set allowCompoundWords settings to false
  • Add a couple of interesting dictionaries
  • Create and use a custom dictionary because our ignoredWords list is too long already
  • Please the check by changing some words

@dgdavid dgdavid requested a review from imobachgs March 14, 2023 16:15
@coveralls
Copy link

Coverage Status

Coverage: 79.96%. Remained the same when pulling aa58ad2 on improve-cspell-config into aaa58ee on master.

@dgdavid dgdavid merged commit 6816725 into master Mar 14, 2023
@dgdavid dgdavid deleted the improve-cspell-config branch March 14, 2023 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants