Skip to content
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

Adapt changes from original fnogatz/xsd2json #1

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

fnogatz
Copy link

@fnogatz fnogatz commented Feb 7, 2015

Hi Alban,

thanks for putting such an effort in making xsd2json better! In order to keep getting updates from the original module I've done some little changes on your fork:

  • 76e2bb8: Adapt new CLI interface of the Prolog library which now consumes filenames instead of XSD strings
  • 6adabe9: In order to natively support xs:include and xs:import it is necessary to prefix all constraints by its filename IName
  • e096ac4 and c226fcc: Some slight changes.

Now your version of the Prolog library is nearly the same as mine in fnogatz/xsd2json@084a437. Your given npm test is still valid.

Because the bug of using type: 'mySelfDefinedType' instead of type: { $ref: '#/definitions/mySelfDefinedType' } has been fixed in the newest version of xsd2json I will try to remove the fixes in json-processing.js. We should also try to adapt the yet implemented xs:include and xs:import support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant