LSP icon indicating copy to clipboard operation
LSP copied to clipboard

Don't attach LSP to syntax test files

Open jwortmann opened this issue 1 year ago • 2 comments

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 avatar Oct 15 '24 16:10 jwortmann

@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?

jwortmann avatar Oct 15 '24 16:10 jwortmann

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

jfcherng avatar Oct 15 '24 16:10 jfcherng