json-schema-spec icon indicating copy to clipboard operation
json-schema-spec copied to clipboard

Support for adding multiple levels of validation errors

Open stevengoossensB opened this issue 2 years ago • 1 comments

As already discussed in #270 , I think there should be functionality to make a distinction between an ERROR (e.g. breaking a build) and a WARNING (something that should be changed in metadata, not blocking problem).

I would add a key errorLevel, with levels "error", "warning", "deprecated" and add this to the validation errors.

stevengoossensB avatar Jan 31 '23 08:01 stevengoossensB

Hi, thanks for bringing this back up. It's been a while and revisiting old topics is often useful.

I'm actually the primary member leading the effort for the specified output formats. You can follow the output label to see relevant issues.

I think my stance as stated in that issue remains unchanged. Essentially, feel that "error level" is an application-level concern. Different applications will have different priorities for the errors, and it's their responsibility to interpret what a validation returns.

Also, with the output format that is now specified, applications have much more information available to make that determination than they did when that issue was active.

I don't think the specification has the purview to determine for everyone which errors can be considered warnings or critical. We just report that something didn't match what was expected and leave interpretation to the consumer.

gregsdennis avatar Jan 31 '23 09:01 gregsdennis

Closing for now as I don't see an action here. Happy to reopen after further discussion.

gregsdennis avatar Jun 18 '24 21:06 gregsdennis