Skip to content

replace original xsd2json with node-xsd2json for PR#5

Closed
albanm wants to merge 62 commits into
fnogatz:masterfrom
MGDIS:master
Closed

replace original xsd2json with node-xsd2json for PR#5
albanm wants to merge 62 commits into
fnogatz:masterfrom
MGDIS:master

Conversation

@albanm

@albanm albanm commented Nov 7, 2014

Copy link
Copy Markdown

Hello,

First of all: I absolutely don't expect you to accept this PR as it is.

It contains a whole new node.js CLI program that wraps around your original prolog program in an attempt to fix the limitations I encountered. You can have a look at the readme for instructions on how to test it.

Most of it you will probably consider dirty workarounds, but I am quite in a hurry to validate the feasibility of XSD->JSON schema for my company. And I need support both for inclusions and type extensions. I will be glad if my code shrinks when your program makes progress.

Of particular interest for you: the test directory contains a small set of XSDs with a good overview of inclusions and extensions test cases.

Thanks

@fnogatz

fnogatz commented Nov 7, 2014

Copy link
Copy Markdown
Owner

Did the first step: I wrote a node wrapper and published the xsd2json module on npm.

@fnogatz

fnogatz commented Nov 11, 2014

Copy link
Copy Markdown
Owner

xs:include and remote XSD given by URLs are now supported in the includes branch. Will merge into master once xs:import is implemented too.

@fnogatz

fnogatz commented Mar 8, 2017

Copy link
Copy Markdown
Owner

Since both versions diverged too much and my initial pull request to adapt the interface changes into the fork got ignored, I will close here. Thank you for putting time into your fork. I hope, sometime both versions are similar in respect of their features :)

@fnogatz fnogatz closed this Mar 8, 2017
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.

5 participants