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

Support workspace symbols request #127

Merged

Conversation

datho7561
Copy link
Contributor

Requires eclipse/lsp4mp#313

Signed-off-by: David Thompson [email protected]

@angelozerr
Copy link
Contributor

Please add a demo and a section with workspace symbols in https://github.com/redhat-developer/vscode-microprofile/blob/master/docs/JavaFeatures.md

@angelozerr
Copy link
Contributor

angelozerr commented Jan 26, 2023

Please add an entry at https://github.com/redhat-developer/vscode-microprofile#microprofile-java-features and please update each entrie with a link to the documentation.

@datho7561 datho7561 force-pushed the 87-endpoint-workspace-symbols branch from 2017ea3 to 12cfc79 Compare January 26, 2023 13:47
@datho7561
Copy link
Contributor Author

Okay. I've added some documentation. Let me know what you think.

README.md Outdated
* [Validation and Quick Fix support for MicroProfile](./docs/JavaFeatures.md#validation-and-quick-fix-support)
* [Code Lens support for MicroProfile](./docs/JavaFeatures.md#codelens-support)
* [Workspace symbols](./docs/JavaFeatures.md#workspace-symbols-support)
* [Code snippets](./docs/JavaFeatures.md#)
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it should be

JavaFeatures.md#snippets-support

@@ -37,3 +37,51 @@ Diagnostics are supported for MicroProfile annotations which targets errors and
On debug, a Quarkus application run with MicroProfile sources supports a URL CodeLens that routes to the endpoint debug path.

![MP Java Code Lens](./res/MPJavaCodeLens.png)

## Snippets support
Copy link
Contributor

Choose a reason for hiding this comment

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

I will move this section just after completion support

@datho7561 datho7561 force-pushed the 87-endpoint-workspace-symbols branch from 12cfc79 to e17e55e Compare January 26, 2023 15:07
@angelozerr angelozerr merged commit 95017fa into redhat-developer:master Jan 26, 2023
@angelozerr
Copy link
Contributor

Thanks @datho7561 !

@angelozerr angelozerr added this to the 0.7.0 milestone Jan 26, 2023
@datho7561 datho7561 deleted the 87-endpoint-workspace-symbols branch January 26, 2023 15:10
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.

2 participants