Skip to content

Clickable InlayHint for Java type#533

Merged
datho7561 merged 1 commit intoredhat-developer:masterfrom
angelozerr:clickable-inlayhint
Aug 17, 2022
Merged

Clickable InlayHint for Java type#533
datho7561 merged 1 commit intoredhat-developer:masterfrom
angelozerr:clickable-inlayhint

Conversation

@angelozerr
Copy link
Copy Markdown
Contributor

Clickable InlayHint for Java type

Fixes #702

Signed-off-by: azerr azerr@redhat.com

@datho7561
Copy link
Copy Markdown
Contributor

Is this PR ready? If so, could you please mark it as ready for review?

@angelozerr angelozerr marked this pull request as ready for review August 17, 2022 14:02
@angelozerr
Copy link
Copy Markdown
Contributor Author

Yes it is ready. I switched to real pr

Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are a few comments that need to be fixed, but otherwise it appears to be good

Comment thread src/qute/languageServer/inlayHintsProvider.ts Outdated
Comment thread src/qute/languageServer/inlayHintsProvider.ts Outdated
@angelozerr angelozerr force-pushed the clickable-inlayhint branch from 8b7a048 to 76b9e97 Compare August 17, 2022 16:18
Fixes redhat-developer#702

Signed-off-by: azerr <azerr@redhat.com>
Copy link
Copy Markdown
Contributor

@datho7561 datho7561 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Thanks, Angelo!

@datho7561 datho7561 added the enhancement New feature or request label Aug 17, 2022
@datho7561 datho7561 added this to the 1.12.0 milestone Aug 17, 2022
@datho7561 datho7561 added the qute label Aug 17, 2022
@datho7561
Copy link
Copy Markdown
Contributor

Requires redhat-developer/quarkus-ls#704

@datho7561 datho7561 merged commit f57b552 into redhat-developer:master Aug 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request qute

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants