Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

perf: don't set ft on otter buffers #41

Merged
merged 4 commits into from
Jun 5, 2023
Merged

perf: don't set ft on otter buffers #41

merged 4 commits into from
Jun 5, 2023

Conversation

jmbuhr
Copy link
Owner

@jmbuhr jmbuhr commented Jun 4, 2023

to avoid other plugins that we don't need attaching to the otter buffers.
Attach language servers manually by calling the callback from the 'lspconfig' autocommand populated by setup on the lspconfigs.

instead, lsp clients that are configured are attached manually
and the otter nvim-cmp sourced is activated on attach.
This prevents other plugins, vim syntax highlighting and treesitter
from unnecessarily attaching to the otter buffer and significantly
speeds up starting.

TODO: Howver, this now leads to multiple servers for those that attach
automatically on ft. Need to start the server **not** in single file
mode
-> need to make this async now
@jmbuhr jmbuhr merged commit 6aa0699 into main Jun 5, 2023
@jmbuhr jmbuhr deleted the no-otter-ft branch June 5, 2023 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant