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

ignoreWords doesn't support hyphenated words. #1497

Closed
2 tasks done
Jason3S opened this issue Aug 9, 2021 · 1 comment · Fixed by #1520
Closed
2 tasks done

ignoreWords doesn't support hyphenated words. #1497

Jason3S opened this issue Aug 9, 2021 · 1 comment · Fixed by #1520
Assignees
@Jason3S
Labels
bug Needs Investigating
Milestone
v-next

Comments

@Jason3S
Copy link
Collaborator

Jason3S commented Aug 9, 2021

Info

Kind of Issue

  • runtime - command-line tools

Which Tool or library

  • cspell-lib -- library that does the actual spell checking.

Which Version

Version: 5.6.6

Bug Description

Related to:
Support ignoring hyphenated/complex terms · Issue #284 · streetsidesoftware/vscode-spell-checker

Hyphenated words in a dictionary are correctly handled, but hyphenated words in ignoreWords are not.

To Reproduce

cspell

    "ignoreWords": [
        "curssor-botcvhed"
    ],

README.md

Some text with curssor-botcvhed

Expected behavior

Expect that curssor-botcvhed is correctly ignored.
@Jason3S Jason3S added this to the v-next milestone Aug 9, 2021
@Jason3S Jason3S removed the new issue label Aug 14, 2021
@Jason3S Jason3S self-assigned this Aug 14, 2021
Jason3S added a commit that referenced this issue Aug 14, 2021
@Jason3S
fix: Make sure hyphenated words work in ignoreWords
df8b3e8 
fix #1497
@Jason3S Jason3S mentioned this issue Aug 14, 2021
Merged
Jason3S added a commit that referenced this issue Aug 14, 2021
@Jason3S
fix: Make sure hyphenated words work in ignoreWords (#1520)
81a86c7 
* fix: Make sure hyphenated words work in ignoreWords
  fix #1497
* Update exclusionHelper.ts
* test: improve code coverage
* test: Update integration test snapshots
@github-actions
Copy link
Contributor

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@Jason3S Jason3S

Labels
bug Needs Investigating
Projects
None yet
Milestone
v-next
Development

Successfully merging a pull request may close this issue.

fix: Make sure hyphenated words work in ignoreWords streetsidesoftware/cspell
1 participant
@Jason3S