Docs: specify that language-server table must be before language and grammar array #7153
Conversation
|
Similarly people have started updating https://github.com/helix-editor/helix/wiki/How-to-install-the-default-language-servers with the new language server config but specificying it below the language table which should probably be fixed as well. It's late here so I will do that in the morning though. |
Co-authored-by: Michael Davis <[email protected]>
|
this might not be the best place to ask this but I wonder if we should put a note on the installing language servers wiki page that we are moving to a new configuration format for language servers and then start changing the wiki to the new format. Or wait till the next release to move the wiki over to the new format. We could still have a note but just saying something like: "The configuration on this page is only for the current release here is how to use it with the current master." |
|
been looking at this again and it seems like you don't actually need to put the language-server table above any arrays of tables, I am guessing something else was happening in those issues and I incorrectly assumed that the moving the |
|
ah this was a problem when adding the new language server when running. then reloading config, and then restarting the server. If it seems possible to fix I'll make an issue for it. |
Due to a limitation in toml, when specifying the new config for language servers from #2507 because they are just tables they must be put before the arrays of tables (language and grammar). I have seen a couple issues and matrix questions pop up where people didn't know this and put the language server configuration after the language config and then nothing happens, #7149 and #7101 (comment). If we think there is going to be more arrays of tables in the future we could make the message more general but I think this way is more clear. I couldn't find any official documentation about this behaviour of toml but I am pretty sure it is the case.