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.

Sign up for GitHub

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

Jump to bottom

if default is present, field counts not required #2

Open

BorisChistov wants to merge 2 commits into fge:master from BorisChistov:master

BorisChistov commented Nov 15, 2016

No description provided.

BCHistov added 2 commits

November 15, 2016 20:49


          if default value in avro schema presents, make this field not required.

e9ec6f6


          Removed trash output from tests

77057e5

snagafritz added a commit to Snagajob/json-schema-avro that referenced this pull request


          Avro fields with default values, or unions that include the null type…

a9659d5

…, should not be included in required properties of JSON schema

Combined fixes in the following PRs from the forked repository:
fge#2
fge#3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet