ajv
ajv copied to clipboard
Extend discriminator to support number and boolean values
What issue does this pull request resolve?
Part of #1663 - as discussed in the thread...
What changes did you make?
Extending discriminator
to support number
and boolean
types.
Is there anything that requires more attention while reviewing?
I have not changed jtd
as I was not even sure if I should or not?
Great - thank you - made some comments
@pkuczynski can you do review changes? it's useful feature
I'd also like to use this feature.
@pkuczynski let me know if you don't have time for it, I can take it form here.
@Froctnow @fredericosilva I just pushed changes requested by @epoberezkin.
@epoberezkin can you have a look again and resolve threads if you are happy with my changes?
@epoberezkin I am not sure where did the build failure came from, as I haven't touched those files. Can you help me to solve it?
Hey, any progress on this? I could really use this feature
@MxtOUT I am still waiting for the feedback from @epoberezkin...
We'd love to get this fix merged, @epoberezkin @pkuczynski any update on this?
I am waiting for the feedback from @epoberezkin...