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

Fix client settings cannot be overridden #4383

Conversation

vljosa-vivodyne
Copy link
Contributor

Create hash table with :test 'equal so that
lsp-register-custom-settings can be called again to override settings.

Resolves #4382

Create hash table with `:test 'equal` so that
`lsp-register-custom-settings` can be called again to override
settings.

Resolves emacs-lsp#4382
@yyoncho
Copy link
Member

yyoncho commented Mar 19, 2024

Thank you!

@yyoncho yyoncho merged commit 8903da3 into emacs-lsp:master Mar 19, 2024
11 of 14 checks passed
@alanz
Copy link
Contributor

alanz commented Mar 20, 2024

FYI, this change causes my semantic tokens to disappear in rust-mode, set up as

(use-package lsp-mode
  :hook (scala-mode rust-mode rust-ts-mode)
  ...
  :custom
  (lsp-semantic-tokens-enable t)
...

If I comment it out, they come back.

@vljosa-vivodyne
Copy link
Contributor Author

@alanz, I'm not able to reproduce. I'm not sure I fully understand how semantic tokens are supposed to work, but I think the pink distance_to in let distance = point_a.distance_to(&point_b); means it's working. (At least that's something that changes with (lsp-semantic-tokens-enable t).)

If I'm wrong, can you please give a piece of example code and explain how to see whether semantic tokens are working so I can try again to reproduce?

image

@alanz
Copy link
Contributor

alanz commented Mar 20, 2024

For rust analyzer, references will show up bold, and mutable items underlined. So making a small change to RA code, I get

Screenshot 2024-03-20 at 15 33 08

ra_fixture is both bold and underlined. If you remove the mut (as it normally is), the underline disappears.

@alanz
Copy link
Contributor

alanz commented Mar 20, 2024

We chose bold and underline to keep things simple for themes

@alanz
Copy link
Contributor

alanz commented Mar 21, 2024

And after the update today it works. And I don't see anything in the commit history that could have done it. It must have been some other part that updated. Sorry for the noise.

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.

Client settings cannot be overridden
3 participants