redhat-developer / quarkus-ls

Language server for Quarkus tooling
Eclipse Public License 2.0
44 stars 15 forks source link

Relaxed plugin validation and errors (false positive) #960

Open ia3andy opened 1 week ago

ia3andy commented 1 week ago

I do believe that it is essential for the Quarkus plugin to be relaxed about raising errors when they are not 100% sure (totally avoiding false positive).

The plugin is here as a helper to make things easier and quicker, the validation is a nice to have (and should be best effort). When there is the question of raising an error, the plugin should:

Anyway Quarkus build is here as backup to make sure everything is fine for the user.

This is a false positive: image

This is a good weak warning: image

ia3andy commented 1 week ago

For the false positive, I think the issue is that the method returns a generic type T which is not accepted by the plugin