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

Spelling.yml filters word partials leading to misspellings not being flagged #894

Open
emteelb opened this issue Oct 16, 2024 · 1 comment · May be fixed by #900
Open

Spelling.yml filters word partials leading to misspellings not being flagged #894

emteelb opened this issue Oct 16, 2024 · 1 comment · May be fixed by #900

Comments

@emteelb
Copy link
Contributor

emteelb commented Oct 16, 2024

Spelling.yml has filter regex terms that filter more words than should be. This leads to words that begin with these regex filter terms to be ignored from the spell check.

Examples:

  • These words should trigger a Vale RedHat.Spelling alert but do not: subcomamnd, Developper, cherrry, et. al.

Here are the filter terms from Spelling.yml relevant to these words:

  - "[cC]he"
  - "[Ss]u"
  - Dev

These are just a few examples. There are likely more filter terms that could prefix other words.

To fix, add a regex word boundary \b after these terms. For example:

  - "[cC]he\b"
  - "[Ss]u\b"
  - Dev\b

Add a regex word boundary after other filter terms that might preface longer words. Possibly consider adding a regex word boundary after every filter term in Spelling.yml, just in case?

I would be happy to submit a pull request for the latter solution.

@emteelb emteelb changed the title Spelling.yml filtering words partials leading to misspellings not being flagged Spelling.yml filters word partials leading to misspellings not being flagged Oct 16, 2024
@aireilly
Copy link
Member

aireilly commented Nov 7, 2024

@emteelb if you would like to submit a PR please do! Would be very much appreciated.

@emteelb emteelb linked a pull request Nov 12, 2024 that will close this issue
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 a pull request may close this issue.

2 participants