-
-
Notifications
You must be signed in to change notification settings - Fork 232
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
False positive on CSpell due to @cspell/dict-win32 not being recognized #2500
Comments
Notably, the issue does not occur when running CSpell directly (e.g., SPELL_CSPELL_PRE_COMMANDS:
- command: "npm install @cspell/dict-win32@2.0.1" Does anyone have any theories as to why CSpell wouldn't be able to import a dictionary when run through MegaLinter? |
Eliminate the need to manually install the CSpell win32 dict locally in order for the VS Code CSpell extension to have access to that dictionary. Keep the manual installation pre-command for now; it is a workaround for oxsecurity/megalinter#2500.
I see you found a workaround, I close the issue :) |
Duplicate #1572 since this is also caused by the fact that CSpell can't see Yarn dependencies when not run via Yarn. This issue would also be addressed by accepting arrays in |
Describe the bug
When we run CSpell through MegaLinter in CI, "nanoserver" and "ltsc", which are words in the CSpell's
win32
dict, come up as false positives despite us having installed and configured thewin32
dict.To Reproduce
Steps to reproduce the behavior:
yarn init
.yarn add --dev @cspell/dict-win32@2.0.1
.Here is a minimum reproduction
Here is the PR where we had this problem: ScribeMD/docker-cache#475
Expected behavior
Expect CSpell to pass on the words "ltsc" and "nanoserver".
Thank you for your help!!!
The text was updated successfully, but these errors were encountered: