Combine messages for minium and maximum handlers

[jdesrosiers]

The minimum/exclusiveMinimum and maximum/exclusiveMaximum handlers can be combined.

{
  "allOf": [
    { "minimum": 5 },
    { "minimum": 7 },
    { "exclusiveMinimum": 7 }
  ]
}

• Expected a number greater than or equal to 5
• Expected a number greater than or equal to 7
• Expected a number greater than 7

This can be collapsed to

• Expected a number greater than 7

This will require combining the minimum, exclusiveMinimum, and draft-04 minimum error handlers an separately the maximum, exclusiveMaximum, and draft-04 maximum error handlers.

[Padmashree06]

I would like to work on this issue .

This is my current understanding, please correct me if I’m wrong anywhere.

In the newer drafts, minimum is an inclusive check (>=) and exclusiveMinimum is an exclusive check (>), both taking numbers.

In draft-04, minimum is still a number but exclusiveMinimum is a boolean. If exclusiveMinimum is true, then the value has to be strictly greater than minimum, otherwise it can be greater than or equal to it.

So even though the encoding is different across drafts, they’re all expressing a lower-bound constraint on a number. When multiple such constraints fail like the above example , my understanding is that we can collapse the errors by keeping only the strongest lower bound, basically the greatest value, and preferring the exclusive one if the values are the same.

Same idea for maximum, except we’d keep the smallest upper bound.

[jdesrosiers]

@Padmashree06 I realized that this needs to be put on hold until #130 is merged so you can incorporate the draft-04 versions of these keywords. You have the right idea, but I'll have you start with the other issue you commented for now.

[jdesrosiers]

@Padmashree06, this is unblocked now. Do you want to do this one next?

[Sam-61s]

Hi @jdesrosiers
Now that the dependency is resolved and the issue is unblocked. If possible, I’d be happy to work on it — please let me know if you can assign it to me

[Padmashree06]

@jdesrosiers I would like to take it up. Please assign it to me.

[jdesrosiers]

Thanks @Padmashree06

[jdesrosiers]

@Sam-61s #134 is available. Would you like to take that one?

[Padmashree06]

Thank you @jdesrosiers