Warning & error suppression for LSP

nushell / tree-sitter-nu

A tree-sitter grammar for nu-lang, the language of nushell

MIT License

121 stars 27 forks source link

Warning & error suppression for LSP #95

Closed CabalCrow closed 6 months ago

CabalCrow commented 6 months ago

This adds several warning & error suppressions for the tsserver lsp. Doesn't actually change any functionality (if you tree-sitter generate you will see that nothing would get changed).

add type docs to functions
supress tsserver errors (they are not actual errors, tsserver just can't properly diagnose the grammar.js)
add _ to unused arguments

fdncred commented 6 months ago

I don't have a problem with this PR but what is tsserver?

CabalCrow commented 6 months ago

It is an lsp (language server protocol) for typescript. I was looking for an LSP to use with grammar.js. I asked in the Matrix channel for tree-sitter what LSP is commonly used to edit that file & was told typescript is the answer. If you check the grammar.js of other tree-sitter projects you will probably also find comments related to tsserver.

fdncred commented 6 months ago

ok, that makes sense that you'd use the typescript lsp for grammar.js. seem benign enough.