"number_type" missing from the suggestions

[celcius112]

Hello, it seems like the number_type validator is missing from the suggestions in https://github.com/ekino/jcv-idea-plugin/blob/master/src/main/kotlin/com/ekino/oss/jcv/idea_plugin/suggestion/ValueToDefaultValidatorsSuggestion.kt.

I could provide a PR (it looks to be a trivial issue), but I heard in my oreillette that there are some big modifications incoming, so it might be better to wait for their merge before resolving this issue.

[leomillon]

Hello @celcius112

What do you mean by "is missing"? Because it is right there: https://github.com/ekino/jcv-idea-plugin/blob/master/src/main/kotlin/com/ekino/oss/jcv/idea_plugin/suggestion/ValueToDefaultValidatorsSuggestion.kt#L96

It is only suggested on JSON Number field values, so if you try this on a String literal value, you won't get it.

[celcius112]

right sorry, I might be blind. however the suggestion still does not appear as expected, for instance:

[leomillon]

Oh right good catch!

This is indeed missing but there: https://github.com/ekino/jcv-idea-plugin/blob/master/src/main/kotlin/com/ekino/oss/jcv/idea_plugin/definition/TemplatedValidator.kt#L70

You can create a quick PR if you want it before the complete rewrite of the plugin (I cannot provide any release date right now).

[leomillon]

Fixed and released in 2.0.1.

Waiting for JetBrains publication approval :)