Skip to content

Enable the SemanticTokensFeature by default #3899

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

Merged
merged 1 commit into from
Apr 8, 2020

Conversation

kjeremy
Copy link
Contributor

@kjeremy kjeremy commented Apr 8, 2020

This is covered under vscode's "editor.semanticHighlighting.enabled"
setting plus the user has to have a theme that has opted into highlighting.

Bumps required vscode stable to 1.44

Closes #3773

This is covered under vscode's "editor.semanticHighlighting.enabled"
setting plus the user has to have a theme that has opted into highlighting.

Bumps required vscode stable to 1.44
@matklad
Copy link
Member

matklad commented Apr 8, 2020

bors r+

🎉

@bors
Copy link
Contributor

bors bot commented Apr 8, 2020

@bors bors bot merged commit 937fd55 into rust-lang:master Apr 8, 2020
@kjeremy kjeremy deleted the enable-semantic-tokens branch April 8, 2020 20:49
@kjeremy
Copy link
Contributor Author

kjeremy commented Apr 8, 2020

Note that we could remove "enableProposedApi": true at this point but I think we should leave it since we'll need it later on.

@matklad
Copy link
Member

matklad commented Apr 8, 2020 via email

@kjeremy
Copy link
Contributor Author

kjeremy commented Apr 8, 2020 via email

@kjeremy
Copy link
Contributor Author

kjeremy commented Apr 9, 2020

I'm going to reverify this after updating @types/vscode to 1.44 but that upgrade breaks vscode-languageclient due to duplicate definitions: microsoft/vscode-languageserver-node#610.

@Geobert
Copy link
Contributor

Geobert commented Apr 13, 2020

Woah, you mean we can use semantic highlighting in stable now? Nice!!

@kjeremy
Copy link
Contributor Author

kjeremy commented Apr 13, 2020

Yes with latest vscode stable and a theme (the default ones work) that supports it.

@Geobert
Copy link
Contributor

Geobert commented Apr 14, 2020

Can I have a link on what a theme needs to havo in order to support this feature? (I have my own theme)

Thank you :D

@kjeremy
Copy link
Contributor Author

kjeremy commented Apr 14, 2020

https://github.com/microsoft/vscode/wiki/Semantic-Highlighting-Overview

@matklad
Copy link
Member

matklad commented Apr 15, 2020

https://github.com/microsoft/vscode/wiki/Semantic-Highlighting-Overview

I wonder if they have like, ... the actual docs for this?

@luiswirth
Copy link

Is semantic highlighting only available on vscode? Or does emacs support it too?

@kjeremy
Copy link
Contributor Author

kjeremy commented Apr 17, 2020 via email

@matklad
Copy link
Member

matklad commented Apr 17, 2020

support is not yet implemented in emacs: emacs-lsp/lsp-mode#1482

lnicola pushed a commit to lnicola/rust-analyzer that referenced this pull request Jan 7, 2025
Android: Added support for prctl handling thread names

Addresses the first part of rust-lang#3618.
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.

Next vscode release (early April): semantic tokens become stable
4 participants