Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix support for $comment keyword, add $comment to schema_test ExampleBasic() #33

Merged
merged 1 commit into from
Dec 20, 2018

Conversation

eponymous301
Copy link
Contributor

Patch to fix issue with schemas containing "$comment" plus add $comment to schema_test.go ExampleBasic()

(they fail with following error message)

unmarshalerDecoder: error unmarshaling $comment from json: json: cannot unmarshal string into Go value of type jsonschema._schema, error found in #10 byte of ...|Raw"

@b5 b5 self-requested a review December 20, 2018 18:50
Copy link
Member

@b5 b5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Thanks so much @eponymous301!

@b5 b5 merged commit 3313399 into qri-io:master Dec 20, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants