Filter for Java (server) snippet

[angelozerr]

Filter for Java (server) snippet

Fixes #265

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

[angelozerr]

To test this PR, open the microprofile-metrics project (used for test), open a Java file, type @ to open completion, you should see only metrics snippets:

Remove metrics dependency and replace with openapi dependency in the pom.xml

In other words, replace:

 <dependency>
      <groupId>io.quarkus</groupId>
      <artifactId>quarkus-smallrye-metrics</artifactId>
    </dependency>

with

 <dependency>
      <groupId>io.quarkus</groupId>
      <artifactId>quarkus-smallrye-openapi</artifactId>
    </dependency>

After refreshing, you should see only openapi snippets:

@rzgry I had to fix your openapi snippets because some type package was wrong. Please check my fix are OK.

[angelozerr]

I need to clean my code and write tests before switching to a real PR.

[xorye]

Works great on my machine.

It wasn't a part of this PR but just one small comment so far:

quarkus-ls/microprofile.ls/com.redhat.microprofile.ls/src/main/resources/com/redhat/microprofile/snippets/mp-openapi.json

Line 106 in 389c184

"\trequired = ${2:true}, ",

Please remove the extra space before the last " so that it becomes "\trequired = ${2:true},",

[angelozerr]

Works great on my machine.

Thanks for the feedback.

Please remove the extra space before the last " so that it becomes "\trequired = ${2:true},",

fixed.

@xorye could you please review my code (javadoc and the README please).

[xorye]

Also, it looks like SnippetContextProperties.java and SnippetContextPropertiesDeserializer.java can be deleted?

[angelozerr]

Also, it looks like SnippetContextProperties.java and SnippetContextPropertiesDeserializer.java can be deleted?

Good catch! Fixed.

[xorye]

Awesome, I'm good with this being merged

[angelozerr]

Thanks @xorye for your great feedback.