-
-
Notifications
You must be signed in to change notification settings - Fork 217
update the test suite for draft-next #725
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
Comments
Can you provide details of misaligned tests that you've found, please? Just saying "it's wrong" isn't particularly helpful. |
draft-next/id.json
that is the only one I checked 😉 I stopped after you mentioned that the tests are not up to date... I can check my other failing tests if it helps. |
That test is dropped by #718 (merged just now). |
I'm down to two test failures (unevaluatedProperties/propertyDependencies) implementing draft-next, so I think there is no "wrong" test anymore. I wonder if there are missing tests though. draft-next talks about
and I expected some failing tests for the object case but there are none.
|
Honestly, I wouldn't worry about getting draft-next right now. It's not published, and there's clearly work to be done. We're going to have to go through it carefully before publication. |
Sounds like we should close this then. If you have specific issues nonetheless (of tests which don't match the spec text), open those please. |
not all tests align with the draft-next schemas
The text was updated successfully, but these errors were encountered: