catch missing schemas

[adrai]

something like this

[mafintosh]

shouldn't it be an error if a referenced schema is missing? we should probably just throw here actually (thoughts? @watson @ebdrup)

[adrai]

error is ok for me :-) just wanted to do like: https://github.com/geraintluff/tv4#missing-schema

[watson]

@adrai normally I would say it's a bug in the schema if a $ref is missing (as opposed to a validation error) and it should therefore throw I guess. But there might be a use-case I'm not considering?

[watson]

Follow up thought: I guess the most important thing is that as a dev I'm not confused about the difference between validation errors and "schema" errors. I know you've created a new property called validate.missing as opposed to validate.errors, but the question is if that's obvious enough? A throw would just make it really obvious. But again, I might not know your use-case.

[adrai]

For my use case it does not really matter… it’s just important to know if a schema is missing…
If you want you can close this pull request and do it like you prefer ;-) You have the choice :-)

Il giorno 24-mar-2015, alle ore 12:34, Thomas Watson Steen <notifications@github.commailto:notifications@github.com> ha scritto:

Follow up thought: I guess the most important thing is that as a dev I'm not confused about the difference between validation errors and "schema" errors. I know you've created a new property called validate.missing as opposed to validate.errors, but the question is if that's obvious enough? A throw would just make it really obvious. But again, I might not know your use-case.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/46#issuecomment-85457944.

[adrai]

What's the proposed decision?

[daditto]

I, too, am eagerly waiting for a decision :-)

[adrai]

What was/is the decision?