-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
some code I added to make OpenApi reader work on my files #26
base: master
Are you sure you want to change the base?
Conversation
I don't get what this should fix. The primitive schema should not be visited because it rather considered to be abstract. As for the OALink this covers existing bugs. Can you provide me with the file you want to read instead? |
So I added a test for each case:
|
I don't get why you make extension methods here for JSONSchema. Look at zweidenker/JSONSchema#40 that adds minItems and maxItems. What else needs to be done? |
Can remove the minItem and json schema stuff please? |
Ok, i just
There should now be 2 test failing, which i think highlights 2 issues |
No description provided.