-
Notifications
You must be signed in to change notification settings - Fork 74
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
display build target info #660
Conversation
42d7047
to
e226987
Compare
src/treeview.ts
Outdated
@@ -191,6 +191,7 @@ class MetalsTreeDataProvider implements TreeDataProvider<string> { | |||
? Uri.parse(item.nodeUri) | |||
: undefined, | |||
collapsibleState: toTreeItemCollapsibleState(item.collapseState), | |||
contextValue: item.contextValue, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this needed in the protocol or can be handle it on the vs code side?
src/treeview.ts
Outdated
@@ -191,6 +191,7 @@ class MetalsTreeDataProvider implements TreeDataProvider<string> { | |||
? Uri.parse(item.nodeUri) | |||
: undefined, | |||
collapsibleState: toTreeItemCollapsibleState(item.collapseState), | |||
contextValue: item.contextValue, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MAybe:
contextValue: item.contextValue, | |
contextValue: if (item.nodeUri.startsWith(projects:) && item.nodeUri.contains("_root_")) "projects" else undefined, |
Something along these lines? (though that is Scala code 😅 )
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You suggestion worked. I've made the change. Adding contextValue to the API is no longer needed.
e226987
to
c100b43
Compare
Languageclient has been released https://github.com/scalameta/metals-languageclient/releases/tag/v0.5.8 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's bump to newest snapshot once https://github.com/scalameta/metals/actions/runs/1806111245 is released and resolve conflicts.
I bumped snapshot version in #836 - there is no need to do this here anymore. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome stuff @Arthurm1!
I wanted this feature so badly that I've resolved conflict myself, hope you don't mind ;)
@kpodsiad No problem - thanks |
see scalameta/metals#3380
requires scalameta/metals-languageclient#331