streetsidesoftware / cspell-dicts

Various cspell dictionaries
Other
223 stars 207 forks source link

Russian dictionary errors #17

Closed iandreyshev closed 4 years ago

iandreyshev commented 7 years ago

Cspell marked these words as incorrect, although they are correct: Средняя, страницу, базовая, успеха, информационная, требующая, внимания, основная, использовании, разработчика, потребуется, классы, помощью, типов, кнопки, Демонстрация

Jason3S commented 7 years ago

@iandreyshev,

Thank you. I am assuming that other Russian words are considered correct. What happens if you restart VS Code?

I was able to get it to work: image

It is very possible that something else is causing an issue.

Jason3S commented 7 years ago

@iandreyshev,

I assumed you were using VS Code, that might not be the case.

For the Russian extension in VS Code, I used both Russian dictionaries in this repository, because I found neither one had enough word coverage. I have not had the time to figure out why the larger dictionary didn't not have what seemed to be common words.

Since I don't read Russian, I had used Russian newspaper samples to test the VS Code Russian extension.

If you can suggest a better open source Russian Hunspell dictionary, that would be very helpful.

iandreyshev commented 6 years ago

@Jason3S,

Our team uses phpStrorm to edit code and cspell module from npm. To run cspell we are using grunt-shell.

cspell.config.json:

{
    "version": "0.1",
    "language": "en",
    "dictionaries": ["russian"],
    "import": ["./node_modules/cspell-dict-russian/cspell-ext.json"],
    "words": [
        "keyframes",
        "nowrap",
        "emsp",
        "разработчику",
        "страницу",
        "базовая",
        "Средняя",
        "дефолтная",
        "успеха",
        "информационная",
        "требующая",
        "внимания",
        "ссылочная",
        "основная",
        "использовании",
        "разработчика",
        "потребуется",
        "классы",
        "помощью",
        "типов",
        "кнопки",
        "Демонстрация"
    ],
    "ignorePaths": [
        "src/img/*"
    ]
}
Jason3S commented 6 years ago

Can you try adding the other Russian dictionary as well:

npm install -SD cspell-dict-ru-ru

cspell.config.json:

{
    "version": "0.1",
    "language": "ru,en",
    "dictionaries": [],
    "import": [
        "./node_modules/cspell-dict-russian/cspell-ext.json",
        "./node_modules/cspell-dict-ru-ru/cspell-ext.json"
    ],
    "words": [
    ],
    "ignorePaths": [
        "src/img/*"
    ]
}
iandreyshev commented 6 years ago

Nice, now it`s work! Thanks!

But I think that in the import line 2 need to use "ru_ru"

"import": [
        "./node_modules/cspell-dict-russian/cspell-ext.json",
        "./node_modules/cspell-dict-ru_ru/cspell-ext.json"
    ]
iandreyshev commented 6 years ago

@Jason3S Can you combine this dicts and post result ?

Jason3S commented 4 years ago

I do plan on combining the dictionaries.

Jason3S commented 4 years ago

@iandreyshev The dictionaries have been combined into cspell-dict-ru_ru.