Results 63 comments of Jason Ian Green

Hi @kelson42 after discussing with EP we have decided that this is not something that we will be fixing within the core AJV library. This problem is due to the...

Hi there, could you put together a [runkit](https://runkit.com/esp/ajv-issue) exhibiting the issue? Try and keep the example as simple as possible.

Judging by the example here https://ajv.js.org/json-schema.html#discriminator your schema is not quite right. Looks like you are missing `propertyName` in `discriminator: {propertyName: "foo"}`.

Thanks for that. So after reviewing things I can see that it is indeed a deliberate restriction and there was actually [a PR a while back](https://github.com/ajv-validator/ajv/pull/1729) to add support for...

Hi @b-wils, I have spoken to EP and this is not a change we're open to exploring at this time. I don't have all the context but there are reasons...

Sorry @hir06 I am having trouble replicating your example. Can you recreate [using this runkit template](https://runkit.com/esp/ajv-issue)? In order to isolate the issue you are seeing it is important to be...

Thanks for the contribution @blottn, I can see what you mean about the original wording. I will take a look at this and see, I'll also have to review the...

I've made some small changes just to get it passing linting and see how the tests run.

Not sure why coveralls is erroring. Will try a clean branch and see if I get the same issue.

This has been moved to #2433 due to coveralls issues.