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

Improvements for documentSymbol #375

Open
jpaju opened this issue Feb 21, 2024 · 0 comments
Open

Improvements for documentSymbol #375

jpaju opened this issue Feb 21, 2024 · 0 comments

Comments

@jpaju
Copy link

jpaju commented Feb 21, 2024

Is your feature request related to a problem? Please describe.

No, purely a feature request.

Describe the solution you'd like

Document symbol request currently does not include constructor parameters (neither for classes or case classes) nor bindings in for comprehension. I think it would make sense to support symbols in both of these contexts.

Classes

Currently when querying for documentSymbols for this:

final case class Person(foo: String):
  val bar = 1

Currently: reported symbols are Person and bar.
Desired: reported symbols are Person, foo and bar.

For comprehensions

for 
  n <- List(...)
yield n

Currently: reported symbols are none
Desired: reported symbols are n

Describe alternatives you've considered

Not really sure if there is any alternatives.

Additional context

No response

Search terms

documentSymbol constructor class for-comprehension

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

No branches or pull requests

1 participant