search

sublimelsp / LSP

Client implementation of the Language Server Protocol for Sublime Text
https://lsp.sublimetext.io/
MIT License
1.66k stars 183 forks source link

Don't attach LSP to syntax test files #2531

Closed jwortmann closed 3 weeks ago

jwortmann commented 1 month ago

Closes #2504

Maybe having this check in is_regular_view is not really accurate (regarding the function name), but I think it is the best place for reusability.

jwortmann commented 1 month ago

@jfcherng

Only very few ones (e.g., LSP-copilot) will work well.

Merging this PR would of course also prevent LSP-copilot to run on syntax test files. I don't know how useful LSP-Copilot is for those files; but since you were the one who opened the linked issue, I assume that you are okay with it?

jfcherng commented 1 month ago

Merging this PR would of course also prevent LSP-copilot to run on syntax test files. I don't know how useful LSP-Copilot is for those files; but since you were the one who opened the linked issue, I assume that you are okay with it?

I even hardly write syntax test :D