-
Couldn't load subscription status.
- Fork 1.1k
Port Inlay hints for name parameters #23375
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
Conversation
d41d2a3 to
5e5a48c
Compare
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.
Great work! I think we can improve it a bit though.
presentation-compiler/test/dotty/tools/pc/tests/inlayHints/InlayHintsSuite.scala
Show resolved
Hide resolved
presentation-compiler/src/main/dotty/tools/pc/PcInlayHintsProvider.scala
Outdated
Show resolved
Hide resolved
presentation-compiler/src/main/dotty/tools/pc/PcInlayHintsProvider.scala
Outdated
Show resolved
Hide resolved
presentation-compiler/src/main/dotty/tools/pc/PcInlayHintsProvider.scala
Outdated
Show resolved
Hide resolved
5bc21a7 to
22ec3a6
Compare
presentation-compiler/src/main/dotty/tools/pc/PcInlayHintsProvider.scala
Outdated
Show resolved
Hide resolved
22ec3a6 to
e217b3e
Compare
e217b3e to
dc14b81
Compare
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.
LGTM!
Add inlay hints for name parameters.
Porting scalameta/metals#7400