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

Add configuration to use new experimental ondisk cache for lsp-kotlin #4138

Merged
merged 3 commits into from
Oct 16, 2023

Conversation

themkat
Copy link
Contributor

@themkat themkat commented Aug 16, 2023

Recently a PR in kotlin-language-server for using an on-disk cache was merged: fwcd/kotlin-language-server#337

While it is merged, it is still considered experimental. The PR I have created here adds configuration to lsp-kotlin to be able to use this new on-disk cache. If no configuration is done, it will just use the old in-memory approach to caching. If one uses the on-disk cache, it is probably wise to configure it for each of your projects (maybe using dir-locals, setq-local in hooks, or something else).

@github-actions github-actions bot added the client One or more of lsp-mode language clients label Aug 16, 2023
clients/lsp-kotlin.el Outdated Show resolved Hide resolved
Copy link
Member

@jcs090218 jcs090218 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Other people might want to review this PR, so let's leave this open for a few days! ;)

@themkat
Copy link
Contributor Author

themkat commented Oct 15, 2023

@jcs090218 . Found out after experimenting that the workspace root is a sensible default. It is also what is currently happening in the vscode-kotlin extension. Added this in my latest commit 🙂

@yyoncho yyoncho merged commit 92503ad into emacs-lsp:master Oct 16, 2023
12 of 14 checks passed
@themkat themkat deleted the lsp_kotlin_ondisk_cache branch October 16, 2023 06:02
nverno added a commit to nverno/lsp-mode that referenced this pull request Oct 16, 2023
* master: (27 commits)
  Rename blacklist to blocklist (emacs-lsp#4173)
  Introduce a function to disable a call for particular method for a server (emacs-lsp#4171)
  Fix `lsp-completion--company-match` freezing emacs-lsp#4192 (emacs-lsp#4193)
  Bump the magik language server to 0.8.1 (emacs-lsp#4195)
  Add configuration to use new experimental ondisk cache for lsp-kotlin (emacs-lsp#4138)
  lsp-erlang: add setting for types on hover (emacs-lsp#4188)
  Make it possible to override lsp-cmake server command (emacs-lsp#4182)
  lsp-erlang: update erlang-language-platform download file names (emacs-lsp#4187)
  clojure: add paredit refactorings
  Fix typo in php-guide.md (emacs-lsp#4183)
  Bump magik language server to 0.8.0 (emacs-lsp#4176)
  Bump elixir-ls version (emacs-lsp#4178)
  Remove unbalanced parenthesis and double-quote in comments (emacs-lsp#4177)
  MDX support (emacs-lsp#4141)
  lsp-erlang: Add semantic token support for erlang-language-platform (emacs-lsp#4166)
  lsp-mode: Fix sending shutdown message (emacs-lsp#4167)
  lsp-erlang: update ELP download file names (emacs-lsp#4165)
  Add Move language client (emacs-lsp#4155)
  Add support for python-lsp-server pyls-isort plugin. (emacs-lsp#4163)
  Updating Crystal LSP (emacs-lsp#4156)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client One or more of lsp-mode language clients
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants