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

flagWords does not mark all flagged words. #1953

Closed
Jason3S opened this issue Nov 4, 2021 · 2 comments · Fixed by #2000
Closed

flagWords does not mark all flagged words. #1953

Jason3S opened this issue Nov 4, 2021 · 2 comments · Fixed by #2000
Labels

Comments

@Jason3S
Copy link
Collaborator

Jason3S commented Nov 4, 2021

I switched to Spell Checker because Spell Right cannot flag words from the dictionary.
Spell Checker does not flag everything here:
image

I would expect it to flag everything except Test and test. Does it split on ' and not flag words shorted than 2 letters?

My .vscode/cSpell.json:

{
    "version": "0.2",
    "language": "en",
    "words": [
        <omitted>
    ],
    "flagWords": [
        <omitted>
        "therefor",
        <omitted>
        "they'd",
        "they'll",
        "they're",
        "they've",
        <omitted>
        "therefor"
    ]
}
MBP:dir user$ npx cspell trace therefor
bash: npx: command not found

Would this work even if I did install nodeJS? Where does it get the cspell package?

Originally posted by @MeAmAnUsername in streetsidesoftware/vscode-spell-checker#1498 (comment)

@Jason3S Jason3S added the bug label Nov 4, 2021
@Jason3S
Copy link
Collaborator Author

Jason3S commented Nov 4, 2021

Related to #1895

@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 Dec 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant