search

elijah-potter / harper

The Grammar Checker for Developers
https://writewithharper.com
Apache License 2.0
884 stars 23 forks source link

Vim spell integration #143

Open imbolc opened 3 weeks ago

imbolc commented 3 weeks ago

Is there any potential harm in mapping userDictPath to the standard spell file? Shouldn't it be a default suggestion?

lspconfig.harper_ls.setup({
    settings = {
        ["harper-ls"] = {
            userDictPath = vim.fn.stdpath("config") .. "/spell/en.utf-8.add",
        },
    },
})
elijah-potter commented 3 weeks ago

For the time being, I don't see any issue with this. Sally forth!

I'm somewhat opposed to making this default behavior, but if it works out well for you and you think it should be included in the docs, feel free to open a PR for it.

jvoisin commented 3 weeks ago

I'm somewhat opposed to making this default behavior

Why not? I think that integration with vim's native spellchecking mechanism is the least surprising behaviour for users, isn't it?